From 200c4eb77bbe85d1efec5beb898b155ecaa02e7c Mon Sep 17 00:00:00 2001 From: Akeshan <59804062+cyberAkeshan@users.noreply.github.com> Date: Fri, 10 Jul 2026 01:06:42 +0200 Subject: [PATCH] Update Cloud UI handover with sidebar fix --- .../HiveGrid Cloud UI Handover 2026-07-10.md | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/03 Quellen und Referenzen/HiveGrid Cloud UI Handover 2026-07-10.md b/03 Quellen und Referenzen/HiveGrid Cloud UI Handover 2026-07-10.md index 3a7c8b1f..b22cff72 100644 --- a/03 Quellen und Referenzen/HiveGrid Cloud UI Handover 2026-07-10.md +++ b/03 Quellen und Referenzen/HiveGrid Cloud UI Handover 2026-07-10.md @@ -6,14 +6,16 @@ recipient: David status: Info / Dev-Stand stand: 2026-07-10 source_repo: HiveGrid-Cloud -source_branch: origin/main -source_commit: c6f96f2be +source_branch: origin/main + fix branch akeshan/fix-subframe-nested-sidebar +source_commit: c6f96f2be + sidebar fix dab70bcf6 --- # HiveGrid Cloud UI Handover 2026-07-10 Kurzfassung: Der Subframe-UI-Nachzug aus dem letzten Dev-PR ist jetzt in `origin/main` von HiveGrid-Cloud gemerged. Der Code nutzt weiterhin die bestehenden Loader, Actions und View-Models; getauscht wurde vor allem die mittlere Screen-Komposition auf `~/ui`-/Subframe-nahe Page-Komponenten. Für David wichtig: D-FORM ist im Vault bereits ein echter Subframe-Review-Draft, P8/P9/P12 sind validierte Screen-Pakete mit Canvas, aber noch ohne Subframe-Links. +Update 2026-07-10: In den neuen Dev-Pages war die generierte Subframe-App-Shell teilweise rekursiv eingebunden. Dadurch erschien die HiveGrid-Sidebar doppelt. Der Fix liegt in HiveGrid-Cloud auf `akeshan/fix-subframe-nested-sidebar`, Commit `dab70bcf6`: `FormsAdminPage` und `AutomationPages` rendern nur noch app-chrome-freien Content und importieren nicht mehr `~/ui/layouts/DefaultPageLayout`. + ## 1. Code-Stand HiveGrid-Cloud Quelle: @@ -34,6 +36,14 @@ Quelle: | Automations-Laufdetail | `/automations/:id/executions/:executionId` | `AutomationExecutionDetailPage` | Subframe-nahe Timeline-/Detail-Fläche | | Automations-Vorlagen | `/automations/templates` | `AutomationTemplatesPage` | Subframe-nahe Vorlagen-Katalog-Fläche | +### Sidebar-Bugfix / App-Shell-Grenze + +- Ursache: `~/ui/layouts/DefaultPageLayout` ist ein generiertes Subframe-Page-Layout mit eigener Beispiel-Sidebar, Header-Navigation und App-Chrome. +- Fehlerbild: Diese Layout-Shell wurde innerhalb der echten HiveGrid-App-Shell gerendert, dadurch doppelte Sidebar mit HiveGrid-Logo/Nav. +- Fix: `FormsAdminPage` und `AutomationPages` verwenden keine `DefaultPageLayout`-Wrapper mehr, sondern nur noch Content-Flächen innerhalb der bestehenden Route-Shell. +- Regression: In HiveGrid-Cloud gibt es dafür den Test `apps/web/app/components/screens/subframe-layout-boundary.test.ts`. +- Lesson für weitere Designs: Subframe-Page-Layouts mit Sidebar/Header nicht direkt in `apps/web/app/components/screens/**` übernehmen. Nur app-chrome-freien Page-Content übernehmen; Dialog-/Drawer-Layouts bleiben für echte Modale/Drawer ok. + ### Was im Code bewusst unverändert blieb - Loader, Actions, Permissions, Owner-Filter und bestehende View-Models bleiben in den Routes/Services. @@ -47,6 +57,7 @@ Quelle: - `pnpm lint` war grün. - `pnpm run verify:i18n` war grün. - Nach dem PR-Fail wurde der Test `apps/web/app/routes/app.$tenant/forms.$id.i18n.test.ts` angepasst und gezielt grün ausgeführt. +- Für den Sidebar-Fix kam `apps/web/app/components/screens/subframe-layout-boundary.test.ts` dazu. ## 2. Dev-Docs und Ticket-Status @@ -141,3 +152,5 @@ Status: `Validiert`, Subframe noch offen. Hey David, kurzer UI-Stand aus HiveGrid-Cloud: Der Subframe-UI-Nachzug ist in `origin/main` gemerged. Neu im Code sind `/forms` als `FormsAdminPage` nach D-FORM `S00` sowie die P12-Verwaltungsflächen `/automations`, Läufe, Laufdetail und Templates über `AutomationPages`. Die bestehenden Loader/Actions/VMs wurden nicht umgebaut, es ist primär ein Screen-Kompositionsswap auf `~/ui`/Subframe-nahe Komponenten. Für dich wichtig: D-FORM ist im Wireframes-Repo jetzt als Review-Draft mit echten Subframe-Links vorhanden. Im Code ist davon bisher nur `S00 Formulare verwalten` übernommen. Der komplette Builder, Public Form, Public States und Antworten-Tabelle sind noch offen. P12 Verwaltung ist im Code teilweise nachgezogen, aber D-FLOW, also der echte Flow-/Node-Editor, bleibt offen und sollte separat behandelt werden. P8/P9/P12 sind im Vault validiert und mit Canvas geplant, aber bis auf D-FORM noch ohne Subframe-Links. + +Nachtrag: Wir haben direkt danach einen kritischen Layout-Bug gefunden. Bitte bei Subframe-Handoffs beachten: `DefaultPageLayout` aus Subframe darf nicht als kompletter Wrapper in HiveGrid-App-Routes landen, weil HiveGrid bereits eine eigene Sidebar/App-Shell hat. Der Dev-Fix entfernt diese rekursive Shell aus `/forms` und den P12-Verwaltungsseiten. Für künftige Übergaben bitte nur den app-chrome-freien Content bzw. echte Dialog-/Drawer-Flächen übernehmen.