Skip to content

Compact the mobile race-dialog header - #185

Merged
shaug merged 1 commit into
mainfrom
scott/issue-174-mobile-dialog-header
Jul 31, 2026
Merged

Compact the mobile race-dialog header#185
shaug merged 1 commit into
mainfrom
scott/issue-174-mobile-dialog-header

Conversation

@shaug

@shaug shaug commented Jul 31, 2026

Copy link
Copy Markdown
Owner

TL;DR

Removes the obsolete mobile stacking rule so race titles wrap beside the existing icon actions without leaving a near-empty header band.

Summary

  • preserve the compact mobile padding while allowing the base flex layout to keep actions top-right
  • add browser-backed coverage at 320, 375, and 414 px using the longest race title
  • verify the sticky header keeps its size and position while the dialog scrolls

Validation

  • uv run pytest tests/test_rendering.py -k "phone_dialog_header or phone_dialog_metrics" -q
  • make check
  • repository-owned review-code-change: clean at e3b60a9

Fixes #174

@shaug
shaug merged commit 5eff0de into main Jul 31, 2026
2 checks passed
@shaug
shaug deleted the scott/issue-174-mobile-dialog-header branch July 31, 2026 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mobile race-dialog header: remove the obsolete stacked actions row

1 participant