Repository navigation
Conversation
…uitknop - Tabstrook: viewport krimpt tot de inhoud (flex 0 1 auto), zodat de "+" direct na het laatste tabblad staat; bij overloop blijft hij aan de eindrand zichtbaar. RTL spiegelt via flex; toetsenbordvolgorde ongewijzigd. - Projectoverzichtskaart: sluitknop flex none (26x26), gap in de kopregel, titelblok minWidth 0 met afbreking; badge en dirty-stip krimpen niet. - Browserspec tabstrip-plus-kaartsluitknop (eerst rood), schermbewijs voor/na. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
De rondleidingskaart zat in dezelfde Suspense-grens als alle lazy dialogen. Laadde een andere dialoog nog (bv. de tutorialvraag), dan verborg React de hele grens met display:none: de kaart werd 0x0 en de knop onklikbaar (CI-flake first-start-tutorial op PR #293). Nu een eigen grens rond TourOverlay. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Nozzit
changed the title
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Twee kleine UI-fouten die de eigenaar zag tijdens de visuele check van #109 (2026-10-07). Hij vroeg om "een agent die dit heel snel meeneemt".
Wat en waarom
flex: 0 1 auto). Bij veel tabbladen krimpt de strook en blijft de "+" zichtbaar aan de rechterrand. De volgorde van de elementen is gelijk, dus toetsenbord en schermlezer werken zoals eerst.flex: noneop de knop, badge en stip;gapin de kopregel; de titel breekt netjes af.Schermbewijs
Bijvangst uit de CI: rondleiding in een eigen Suspense-grens
De CI faalde twee keer op
first-start-tutorial.spec.ts: de rondleidingskaart was soms 0 × 0 px en dus onklikbaar. Oorzaak:<TourOverlay />stond met alle lazy geladen dialogen in één gedeelde<Suspense>; laadde een andere dialoog in die groep nog (bijv. de tutorialvraag), dan verborg React de hele groep metdisplay:none, de rondleiding inbegrepen. Fix:TourOverlaykrijgt een eigen<Suspense fallback={null}>(src/App.tsx, commit 6579297). Het is een timingfout die al bestond (lokaal 1 op 50 rood vóór de fix); de tabstrookwijziging veranderde vermoedelijk alleen de timing in CI. Na de fix: tutorial-spec en tabstrook-spec elk × 8 groen (96/96),npm run verifyexit 0.Hoe getest
tests/browser/tabstrip-plus-kaartsluitknop.spec.tsmet echte klikken. Vóór de fix rood: "+" 1204 px van het laatste tabblad, knop 15 px breed. Na de fix groen: "+" binnen 8 px, bij 16 tabbladen zichtbaar en klikbaar; knop 26 × 26 px en ≥ 4 px van de badge.npm run verifyzonder corpus: exit 0 (407 browsertests).Gebouwd door een Opus 5.5-agent; schermafbeeldingen door de orkestrator bekeken.
🤖 Generated with Claude Code