Skip to content

fix(web): fold the secondary toolbar actions to icons before every label - #313

Merged
kqesar merged 2 commits into
masterfrom
fix-toolbar-fold
Oct 20, 2026
Merged

kqesar merged 2 commits into
masterfrom
fix-toolbar-fold

Conversation

@kqesar

@kqesar kqesar commented Oct 20, 2026

Copy link
Copy Markdown
Owner

At 1440 px the toolbar fell to all icons since Share joined it, leaving ~550 px empty. A new secondary-icons form folds Export, Import, Share, Stream view and Shortcuts first; Mode, New game, Autotracking, the map layout and big keys keep their labels. Super Metroid and SMZ3 follow the same order.

ALttPR forms now: labels from ~1530 px, secondary-icons ~1130-1520, icons ~860-1130, More below. Checked with unit specs, screenshots at 390/820/1280/1440/1920 (SM and SMZ3 too), lint, types, coverage, build and the e2e suite.

The first fold dropped all labels at once, so adding Share left a bare icon bar at 1440 px.
Export, Import, Share, Stream view and Shortcuts now fold first; Mode, New game and
Autotracking keep their labels until the whole bar must go to icons. Super Metroid and SMZ3
follow the same order.
@github-actions

Copy link
Copy Markdown
Contributor

🔍 CI Results

Check Status
Linting ✅
Type-check ✅
Tests ✅

📊 Test Coverage

Metric Coverage Status
Lines 98.93% 🟢
Statements 98.27% 🟢
Functions 98.21% 🟢
Branches 93.1% 🟢

✅ All checks passed!


Updated at 2026-10-20T19:55:40.673Z

@kqesar
kqesar merged commit ca1377b into master Oct 20, 2026
15 checks passed
@kqesar
kqesar deleted the fix-toolbar-fold branch October 20, 2026 19:57
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.

1 participant