Update Cloud UI handover with sidebar fix #3
@@ -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.
|
||||
Reference in new issue
Block a user