Phone header: Open navigation is below the 44 px touch target (test_adaptive_matrix fails on main)

Chiusa
  • Qualcosa non va?
#478 2 commenti 0 reazioni 1 assegnatario Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@Zamojski5 ci sta già lavorando.

Dal 11/10/2026.

Valutazione

Questa issue non è ancora stata valutata.

Descrizione

./scripts/check_ui.sh test_adaptive_matrix fails on main. This is a pre-existing defect, reproduced on main 2175d249 and while reviewing #380 at 6ce6f81e, a PR that does not touch the header or touch targets.

  • test_01_the_same_journey_works_at_every_fixture_width fails at 8 phone and tablet widths: 320×568, 360×640, 360×800, 390×844, 412×915, 430×932, 640×360 and 600×960.
  • test_06_the_same_journey_in_the_dark_theme fails at 390×844.
  • Assertion: "Open navigation … 44 px touch target … across 42" — the Open navigation control is below the 44 px coarse-pointer target.

Owner: claude-maurycy (#151 adaptive layouts).

Acceptance:

  1. Open navigation meets 44×44 px on coarse pointers at every fixture width, in light and dark.
  2. test_adaptive_matrix passes in Chromium.
  3. A negative control fails without the fix.
Lingua principale
TypeScript
Stelle
3
Fork
0
Merge medio
18h 41m
PR unite (30g)
330

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di ColdPhase/flux

Tutte le issue di ColdPhase/flux

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.