# Layout Review Changelog

Append-only log of all layout review iterations. Each entry records what was captured, what was found, and what was fixed. Newest entries at the top.

---

## Iteration 16 — 2026-04-18
- **Type:** Capture only (no analysis)
- **Devices:** all 7 (portrait + landscape = 14 variants)
- **Screens:** all 12 per device per orientation
- **Screenshots:** 471 total refreshed in-place across `screenshots/current/`
  - Portrait: small-se 27, std-galaxys24 23, std-pixel8 22, large-iphone14max 21, fold-zfold6-inner 27, tablet-ipadmini 24, tablet-ipadpro129 26 (170 total)
  - Landscape: small-se-land 56, std-galaxys24-land 60, std-pixel8-land 51, large-iphone14max-land 49, fold-zfold6-inner-land 29, tablet-ipadmini-land 31, tablet-ipadpro129-land 25 (301 total)
- **Run notes:**
  - Mid-run APK rebuild required — the installed build was older than the `report2.0` commit which added `react-native-view-shot` (native dep). Rebuilt via `expo run:android` and restored capture.
  - Metro stalled once during landscape pass; resolved by Brett. Landscape pass re-ran from small-se-land cleanly after restart.
  - One stray `03_preflop_result_nav_error.png` from the pre-rebuild aborted attempt was cleaned up in small-se.
- **Note:** Screenshots refreshed; `FIX_PLAN.md` / `fix_plan.json` from iteration 15 left in place (not regenerated in capture-only mode). Gallery rendered in no-findings mode.

---

## Iteration 15 — 2026-04-17
- **Type:** Full recapture with exclusions — portrait + landscape, 7 devices each
- **Portrait excluded:** Read ready, Best ready, Custom Build, Settings, Poker Guide (result states of Read/Best still captured)
- **Landscape excluded:** Main Menu, Custom Build, Settings, Poker Guide
- **Portrait captured:** 01 Main Menu, 02 Preflop Ready, 03 Preflop Result, 04 Live Ready, 10 Hand History, 11 Read Result, 12 Best Result (7 screens × 7 devices)
- **Landscape captured:** 02 Preflop Ready, 03 Preflop Result, 04 Live Ready, 05 Read, 06 Best, 10 Hand History, 11 Read Result, 12 Best Result (8 screens × 7 devices)
- **Screenshots:** 143 new captures (plus preserved older shots for excluded screens)
- **Issues:** 10 (P0: 0, P1: 5, P2: 2, P3: 3) — 0 auto-fix, 10 needs-review
- **Resolved this run (confirmed fixed on new screenshots):**
  - Iter 14 #1 (3rd stacked action button clipped on short-landscape phones) — confirmed fixed on small-se-land
  - Iter 14 #2 ipadmini-land portion (landscape tablet dead space) — confirmed fixed
- **New P1 findings:**
  - Live postflop 2x2 buttons clipped at viewport bottom on small-se
  - Live POT pill overlaps community cards on fold-zfold6-inner (near-square portrait)
  - Live 4-button row text clipping on fold-zfold6-inner-land
  - Preflop action history clipped left edge on tablet-ipadpro129 portrait
  - Live action history clipped right edge on tablet-ipadpro129 portrait
- **Still present (recurring):**
  - Iter 14 #2 on tablet-ipadpro129-land (6 screens: 02/04/05/06/11/12 dead space + table clip)
  - Iter 14 #4 on std-galaxys24-land (table bottom arc clipping)
  - Iter 14 #5 on tablet-ipadmini-land (Best Result right column dead space)
- **Pipeline observation (P3):** Live deep link does not reset state between captures — exposed real layout bugs but not reproducible
- **Skill did not modify any source files** (Phases 0-7). Review FIX_PLAN.md and approve or say "apply all fixes and recheck" for Phase 8.

---

## Iteration 14 — 2026-04-16
- **Type:** Landscape re-capture (all 12 screens, all 7 landscape devices — portrait retained from iter 13)
- **Devices:** 7 landscape (small-se-land, std-galaxys24-land, std-pixel8-land, large-iphone14max-land, fold-zfold6-inner-land, tablet-ipadmini-land, tablet-ipadpro129-land)
- **Screenshots:** 306
- **Issues:** 5 (P0: 0, P1: 1, P2: 2, P3: 2) — 0 auto-fix, 5 needs-review
- **Resolved this run (confirmed fixed on new screenshots):**
  - Iter 13 #2 (history "LJ opens to 2.5bb" clip on ipadpro-land)
  - Iter 13 #3 (postflop bet buttons ~80x30dp compressed on ipadmini-land)
  - Iter 13 #4 fixed on ipadmini-land only (tiles now edge-to-edge); still_present on ipadpro129-land
  - R3-3 (EV label truncation "Call→Ca", "3-Bet→3-") continues fixed across all landscape devices
- **New P1 finding:** 3rd stacked action/bet button clipped below viewport on short-landscape phones (small-se-land, std-pixel8-land Live postflop)
- **Still present:** Iter 13 #1 (landscape tablet 4-col body + 50/50 split don't flex-grow vertically); Iter 13 #4 partial (Main Menu tiles only ~40% of 1366dp width on ipadpro129-land)

---

## Iteration 13 — 2026-04-16
- **Type:** Full baseline run (first v3 persistent-folder run, portrait + landscape, all 12 screens)
- **Devices:** all 7 portrait + all 7 landscape (14 device passes)
- **Screenshots:** 391
- **Issues:** 4 (P0: 0, P1: 0, P2: 4, P3: 0) — 0 auto-fix, 4 needs-review
- **Resolved:** R3-3 (EV label truncation) confirmed fixed
- **Still present:** R3-4 (landscape tablet body vertical fill), history text clipping on ipadpro-land, Live bet buttons compressed on ipadmini-land, Main Menu tile width on landscape tablets
- **Portrait baseline:** All 7 portrait devices x 12 screens = clean, no findings
- **New screens captured:** 07 Custom Build, 08 Settings, 09 Poker Guide, 10 Hand History (first time in review)

---

## Iteration 12 — 2026-04-15 (run4)
- **Type:** Full run (landscape only, screens 01-06,11,12)
- **Devices:** all 7 (landscape)
- **Screenshots:** 63
- **Issues:** 8 (P1: 4, P2: 3, P3: 1) — 0 auto-fix, 8 needs-review
- **Fixes applied:** 7 of 8 from run3 fix plan (Issue #4 skipped — preflop_result overhaul in progress)
- **Applied:** reactive ?result=1 effect on Read+Best, Read portrait felt height cap, Train+Live landscape paddingRight, landscapeTableAspect square on tall tablets, Read+Best lFelt/lTable aspect, Train landscape history numberOfLines+adjustsFontSizeToFit
- **REQUIREMENTS.md:** 7 new Feedback History entries added

## Iteration 11 — 2026-04-15 (run3)
- **Type:** Full run (landscape only, screens 01-06,11,12)
- **Devices:** all 7 (landscape)
- **Screenshots:** 60
- **Issues:** 8 (P1: 3, P2: 3, P3: 2) — 1 auto-fix, 7 needs-review
- **Key findings:** Main Menu tile labels wrap mid-word on wide tablets (P1), Live ready landscape 4-col broken on fold (P1), ResultPanel right-edge clipping (P1), felt oval on fold needs Math.min, landscape tablets have vertical dead space
- **Note:** Pruned 2026-04-14/run1 at Phase 0 (both findings applied)

## Iteration 10 — 2026-04-15 (run2)
- **Type:** Full run (landscape only, all screens)
- **Devices:** all 7 (landscape)
- **Screenshots:** 232
- **Issues:** 9 (P1: 1, P2: 6, P3: 2) — 1 auto-fix, 8 needs-review
- **Note:** Most P0/P1 landscape issues from run1 resolved. Remaining work is layout polish. Skill helpers updated mid-run (Metro start in Phase 0, fixed swipe margins in capture_helpers.sh)

## Iteration 9 — 2026-04-15 (run1)
- **Type:** Full run (both portrait + landscape, all screens)
- **Devices:** all 7 portrait + all 7 landscape (14 configs)
- **Screenshots:** 357
- **Issues:** 18 (P0: 3, P1: 4, P2: 8, P3: 3) — all needs-review
- **Note:** First landscape layout review. iPad Pro landscape had broken max-width constraints (P0), ReadScreen/BestScreen card circles overflow in landscape (P1)

## Iteration 8 — 2026-04-14 (run1)
- **Type:** Full run (portrait, screens 01-07,10)
- **Devices:** all 7
- **Screenshots:** 101
- **Issues:** 2 (P1: 1, P2: 1) — 2 auto-fix
- **Note:** Skipped Settings/Poker Guide. Main Menu recaptured with v2 animation-aware detector. Both findings resolved in-session (BestScreen unified below-table label, LiveScreen postflop 2x2 threshold raised to 500dp)

## Iteration 7 — 2026-04-13 (run5)
- **Type:** Full run (portrait, screens 01-07,10)
- **Devices:** all 7
- **Screenshots:** 99
- **Issues:** 5 (P1: 1, P2: 4)

## Iteration 6 — 2026-04-13 (run4)
- **Type:** Full run (portrait, screens 01-07,10)
- **Devices:** all 7
- **Screenshots:** 100
- **Issues:** 5 (P1: 3, P2: 0, P3: 2) — 2 auto-fix, 3 needs-review

## Iteration 5 — 2026-04-13 (run3)
- **Type:** Full run (portrait, excludes Live/Settings/Guide)
- **Devices:** all 7
- **Screenshots:** 77
- **Issues:** 2 (P1: 2)

## Iteration 4 — 2026-04-13 (run2)
- **Type:** Full run (portrait, all screens)
- **Devices:** all 7
- **Screenshots:** 144
- **Issues:** 7 (P1: 2, P2: 4, P3: 1) — 2 auto-fix, 4 needs-review

## Iteration 3 — 2026-04-13 (run1)
- **Type:** Full run (portrait, all screens)
- **Devices:** all 7
- **Screenshots:** 143
- **Issues:** 7 (P2: 5, P3: 2) — 2 auto-fix, 5 needs-review
- **Note:** Best screen recaptured on 4 devices after nav_to verification improvements. Tablet pre-captured by user

## Iteration 2 — 2026-04-12 (run3)
- **Type:** Full run (portrait, all screens)
- **Devices:** all 7
- **Screenshots:** 100
- **Issues:** 6 (P0: 1, P2: 4, P3: 1) — 1 auto-fix, 5 needs-review
- **Note:** tablet-ipadmini and tablet-ipadpro129 had touch bug (screens 02-10 skipped)

## Iteration 1 — 2026-04-12 (run1)
- **Type:** Full run (portrait, all screens)
- **Devices:** all 7
- **Screenshots:** 181
- **Issues:** 5 (P1: 1, P2: 3, P3: 1) — 1 auto-fix, 4 needs-review
- **Note:** First layout review run. Nav failures on fold/tablet devices