Selaa lähdekoodia

TASKS.md: P2-9-bijvangst (dode terug-knop Home) afgerond en verwijderd; P1-13 builder-fix uitgevoerd maar export-sync-blocker gevonden en gedocumenteerd

bob 1 kuukausi sitten
vanhempi
commit
83b0598bca
1 muutettua tiedostoa jossa 95 lisäystä ja 39 poistoa
  1. 95 39
      TASKS.md

+ 95 - 39
TASKS.md

@@ -1,11 +1,23 @@
 # Uitgaanskrant — takenlijst
 
-Bijgewerkt: 2026-08-09 avond (Claude, builder + emulator-sessie). Zie
-`CLAUDE.md` voor werkinstructies/conventies. Elke openstaande taak
-hieronder is zelfstandig te begrijpen zonder de chat gelezen te hebben
-waarin hij ontstond.
-
-**Deze sessie (2026-08-09, avond):** twee taken opgepakt.
+Bijgewerkt: 2026-08-10 (Claude, builder-sessie). Zie `CLAUDE.md` voor
+werkinstructies/conventies. Elke openstaande taak hieronder is
+zelfstandig te begrijpen zonder de chat gelezen te hebben waarin hij
+ontstond.
+
+**Deze sessie (2026-08-10):** twee taken opgepakt. **P2-9-bijvangst
+(dode terug-knop Home) volledig afgerond** — Remove Widget-actie,
+bevestigd via verse export. **P1-13: builder-fix uitgevoerd
+(Expanded+Shrink Wrap uit+Scrollable aan op de StaggeredView-node) maar
+niet doorgezet naar de export** ondanks correcte builder-UI-status
+("Synced", geen foutmelding, blijft staan na reload) — bevestigd met 3
+onafhankelijke verse `flutterflow export-code`-runs. Sanity-check
+bevestigt dat de exportpijplijn zelf werkt (P0-6/P2-9 tonen wél
+correct) — dit lijkt een specifiek sync-probleem bij deze
+property-combinatie, zie P1-13 voor het volledige verslag en het
+verzoek aan Bob om zelf te verifiëren.
+
+**Vorige sessie (2026-08-09, avond):** twee taken opgepakt.
 **P1-19:** builder-fix geprobeerd (Row → Wrap Widget), maar de
 menu-klik registreerde niet (4 pogingen, geen schade) — teruggegeven
 aan Bob met exact stappenplan. **P1-13:** eindelijk de al maanden
@@ -545,9 +557,10 @@ RangeError op deze en de `horecagelegenheidCurrent`-pagina bij een
 niet-bestaande `nid` is al gedekt door P1-1 punt 4 — live bevestigd,
 zie daar.)
 
-**P1-13 · Eigenaar: Claude — bezig (2026-08-10).** Root cause + fix nu
-bekend, builder-werk resteert. Op `HorecagelegenhedenOverzicht` → tab "Activiteiten"
-overflowt de kaartjesgrid. **Root cause definitief bevestigd
+**P1-13 · Eigenaar: Onbepaald — builder-fix toegepast maar NIET
+bevestigd in de export, zie blocker hieronder vóór verder bouwen.**
+Op `HorecagelegenhedenOverzicht` → tab "Activiteiten" overflowt de
+kaartjesgrid. **Root cause definitief bevestigd
 (2026-08-09, Claude, verse volledige stack trace + visuele bevestiging
 op emulator-5554):**
 ```
@@ -572,24 +585,70 @@ kleinere "on the right"-overflows elders in eerdere sessies (los
 bevestigd deze sessie op een ander component: zie P1-19's
 zustervondst op `PUitgaanSliderKaartComponent:179`, zelfde
 "kale-Row/Column-zonder-wrap"-familie maar niet dezelfde plek).
-- **Fix (builder, groter dan een 1-klik-property-wijziging — wél
-  volledig mechanisch/geen ontwerpkeuze):** op
-  `HorecagelegenhedenOverzicht` → Widget Tree → de `Column` (regel 286,
-  binnen de eerste `TabBarView`-tab) → de `FutureBuilder`/`MasonryGridView`-
-  boom **in een `Expanded` wrappen** (zodat de grid een bepaalde hoogte
-  krijgt binnen de Column) **en** op de `MasonryGridView` zelf
-  `Shrink Wrap` uitzetten + physics terug naar de standaard (scrollbare)
-  waarde i.p.v. `Never Scrollable` — waarmee de grid zijn eigen
-  onafhankelijke scroll krijgt. Alternatief, mogelijk eenvoudiger in de
-  builder: de hele Column-inhoud wrappen in een `SingleChildScrollView`
-  (dan mag `shrinkWrap: true` + `NeverScrollableScrollPhysics` blijven
-  staan, zoals dat elders in dit project vaker voorkomt) — welke van de
-  twee makkelijker te zetten is in de builder-UI moet ter plekke
-  blijken. **Nog niet geprobeerd door Claude** (deze sessie ging op aan
-  het vangen van de stack trace zelf) — volgende sessie kan direct met
-  deze twee opties starten. Waarschijnlijk hetzelfde patroon op de
-  andere tabs van deze pagina (Cultuur, Eetgelegenheden, …) — bij het
-  oppakken meteen alle tabs checken, niet alleen Activiteiten.
+- **Bestandspad-correctie (2026-08-10):** de hierboven genoemde
+  `horecagelegenheden_overzicht_widget.dart` bestaat niet meer als
+  zodanig — Bob's project splitst deze pagina inmiddels in
+  `HorecagelegenhedenOverzicht` (dunne `ConditionalBuilder`-wrapper,
+  `[If/Then/Else 2 Conditions]`) die naar
+  `horecagelegenheden_overzicht_page_data_type_widget.dart` of
+  `..._sort_page_widget.dart` doorschakelt. De Activiteiten-tab-Column
+  zit nu op regel ~277 van het `PageDataType`-bestand (structuur/getallen
+  overigens identiek: `Column(mainAxisSize: MainAxisSize.max)` →
+  `FutureBuilder` → `MasonryGridView.builder(shrinkWrap: true, physics:
+  const NeverScrollableScrollPhysics())`). Navigeer in de builder dus
+  naar `?page=HorecagelegenhedenOverzicht` (niet naar de
+  `PageDataType`/`SortPage`-varianten los) — de widget-tree toont dan
+  gewoon de juiste, actieve tak.
+- **Fix uitgevoerd in de builder (2026-08-10, Claude) — mechanisch,
+  bevestigd correct qua stappen, maar NIET bevestigd in de export (zie
+  blocker hieronder):** op de `Column` binnen `TabBar Page` (kind van
+  `TabActiviteiten`) bleek de Column zelf al binnen een bestaande
+  `Expanded(TabBarView(...))` te zitten (dus al bounded-height) — een
+  losse "Scrollable"-toggle op die Column zelf gaf zelfs een expliciete
+  **"Invalid Action"-foutdialoog** terug ("You can't set Expanded for a
+  widget if it's in a Column that doesn't have a defined Height. This
+  includes a Scrollable Column.") toen ik daarna ook Expanded op het
+  kind probeerde — nuttige bevestiging dat FlutterFlow deze combinatie
+  zelf detecteert en terugdraait. **Werkende combinatie (op de
+  `StaggeredView`-node zelf, kind van de Column, niet op de Column
+  zelf):** rechterpaneel → **Expansion → Expanded** (rechtse van de 3
+  iconen: None/Flexible/Expanded, in die volgorde — geen tooltip nodig
+  om te onthouden) + **Shrink Wrap → uit** + **Scrollable → aan**. Geen
+  foutdialoog bij deze combinatie, paneel toont "Synced"/"Saved",
+  waarde blijft staan na page-reload binnen dezelfde sessie.
+- **⚠️ BLOCKER — bevestigd niet-doorzettend naar de export (2026-08-10,
+  Claude):** ondanks bovenstaande drie toggles zichtbaar correct in de
+  builder (en blijvend zo na reload) én "Synced"-status zonder
+  foutmelding, lieten **drie onafhankelijke verse
+  `flutterflow export-code --as-debug`-runs** (over ca. 10 minuten,
+  dezelfde opzet als Bob's eigen `ff-run-fvm.sh` gebruikt) geen enkele
+  wijziging zien: `Column`/`MasonryGridView` in het geëxporteerde
+  bestand staan nog altijd exact zoals vóór de builder-edit
+  (`shrinkWrap: true`, `NeverScrollableScrollPhysics`, geen nieuwe
+  `Expanded`). **Sanity-check uitgevoerd:** de exportpijplijn zelf werkt
+  wél correct — dezelfde export toonde de al langer bevestigde
+  P0-6-hintText-fix (`d507d3b9`/`t8h5f8ir` in
+  `internationalization.dart`) gewoon goed. Ook een heel andere,
+  eenvoudiger wijziging in dezelfde sessie (P2-9's dode terug-knop op
+  Home, kale "Remove Widget"-actie) **wél** correct in de export
+  terechtgekomen — dus dit is geen algemene sessie-brede exportstoring,
+  het lijkt specifiek aan deze combinatie van property-toggles
+  (Expansion/Shrink Wrap/Scrollable op een `StaggeredView` diep genest
+  in een `TabBarView`) gebonden. **Voor Bob: graag zelf in je eigen
+  browser bevestigen of de 3 toggles op `HorecagelegenhedenOverzicht` →
+  Widget Tree → `TabActiviteiten` → `TabBar Page` → `Column` →
+  `StaggeredView` daadwerkelijk staan zoals hierboven beschreven, en zo
+  ja, gewoon een verse `ff-run-fvm.sh`/export proberen** — het kan een
+  eenmalige sync-hik zijn die met een handmatige trigger (of gewoon wat
+  meer tijd) alsnog doorkomt. Zo niet: dit is het eerste bevestigde
+  geval waarbij de builder een wijziging laat zien die structureel niet
+  naar de exportbron doorzet, wat relevant is voor elke toekomstige
+  property-edit-taak, niet alleen deze.
+- **Resterend werk zodra de sync bevestigd/opgelost is:** dezelfde 3
+  toggles herhalen op de overige tabs (Cultuur, Eetgelegenheden,
+  Overnachten, en nog 2 verder niet genoemde tabs — 6 in totaal per
+  `_model.tabBarController` se `length: 6`) — nu nog niet gedaan, geen
+  zin om te herhalen vóór de sync-blocker opgelost is.
 - **Reproductierecept dat de race met Home's eigen overflow (P1-19)
   vermeed (2026-08-09, definitief werkend — vervangt eerdere
   R+deep-link-pogingen):** `flutter run` **direct starten met
@@ -872,18 +931,15 @@ groot dat een nieuwe gebruiker de app na 1x openen niet snapt/niet
 terugkomt. Nog geen concrete builder-fix uitgewerkt — eerst met Bob
 bespreken wat voor onboarding gewenst is (korte welkomsttekst? meteen
 naar provincie/gemeente-keuze i.p.v. Home?) vóórdat dit gebouwd wordt.
-- **Bijvangst, zelfde verkenning — Eigenaar: Claude — bezig
-  (2026-08-10), losstaand van de rest van P2-9 hierboven (die blijft
-  wachten op onboarding-overleg met Bob):** Home's AppBar heeft een
-  hardcoded terug-pijl-knop (`Icons.arrow_back_outlined`, roept
-  `context.pop()` aan) naast het hamburger-menu-icoon
-  (`lib/uitgaanspaginas/home/home_widget.dart`, rond de
-  `FlexibleSpaceBar`). Op de root-pagina is er niets om naar terug te
-  gaan — live getest, de knop doet niets (geen crash, stille no-op),
-  maar oogt verwarrend/onafgemaakt op het allereerste scherm. Kleine
-  fix: knop weghalen op Home, of `automaticallyImplyLeading`/
-  conditie gebruiken zodat hij alleen verschijnt op pagina's waar
-  terug-navigeren zinvol is.
+- ~~Bijvangst, zelfde verkenning: dode terug-pijl-knop in Home's
+  AppBar~~ — **afgerond (2026-08-10, Claude).** `IconButtonBack`
+  verwijderd via Widget Tree → rechtsklik → "Remove Widget" (op
+  `HomeWidget` → `AppBar` → `Row`, naast `IconButtonDrawer`). Bevestigd
+  via verse `flutterflow export-code`: geen enkele referentie meer aan
+  `IconButtonBack`/het bijbehorende `Icons.arrow_back_outlined`-icoon in
+  `home_widget.dart` — de AppBar-Row bevat nu alleen nog de
+  hamburger-menuknop. Geen builder-sync-problemen bij deze simpele
+  Remove-Widget-actie (i.t.t. P1-13's multi-property-toggle, zie daar).
 
 **P2-10 · Eigenaar: Onbepaald.** "Vanavond in [gemeente/provincie]"-
 herinnering (pushmelding of in-app-widget) — uitgelicht uit de