Repository navigation
fix(composer): normalize responsive effort icons - #210
Conversation
Squashed from origin/fix/effort-icon-sizing (0a367f6) onto upstream/main.
Squashed from origin/fix/effort-icon-sizing (0a367f6) onto upstream/main.
|
Follow-up: the effort menu now keeps one full-width level per row on mobile as well as desktop. Removed only the mobile flex-direction, flex-wrap and automatic button-width overrides; icon sizes and selection behavior remain unchanged. The full-width assertion failed before the change and passed after removing those exact overrides in Chromium at 320, 390, 640, 768 and 1280px. Selection remained functional. Local lint and production build passed (existing warnings unchanged). This update is not deployed yet. |
|
Screenshot-based sizing follow-up: mobile/narrow toolbar effort icons are now 18px instead of 20px, and the Slow snail is enlarged from its actual 11px size to 14px. Menu icons remain 16px on mobile and 14px on desktop; full-width menu rows are preserved. Standard desktop toolbar sizes are unchanged. Browser preview checks passed at 320, 390, 640 and 1280px, including the High-level circle; vertical alignment stayed centered and the Slow button stayed 28x28px. Local lint and production build passed with the same existing warnings. These pending UI updates are not deployed yet. |
Squashed from origin/fix/effort-icon-sizing (fa2d4f9) onto upstream/main.
|
The unchanged-code validation retry https://github.com/kahme247/ompweb/actions/runs/37263849206 passed on Linux and Windows. The earlier Linux failure was in Next Google-font loading. The current integration release 9559aa0 includes this PR at fa2d4f9. Verified the actual public UI after deployment: at 390px, toolbar effort 18px and Slow snail 14px, with menu icons still 16px; at 1280px, existing toolbar sizes and 14px menu icons are unchanged. Every menu option uses its full row width. |
Squashed from origin/fix/effort-icon-sizing (fa2d4f9) onto upstream/main.
Squashed from origin/fix/effort-icon-sizing (fa2d4f9) onto upstream/main.
Squashed from origin/fix/effort-icon-sizing (fa2d4f9) onto upstream/main.
Squashed from origin/fix/effort-icon-sizing (fa2d4f9) onto upstream/main.
Squashed from origin/fix/effort-icon-sizing (fa2d4f9) onto upstream/main.
Squashed from origin/fix/effort-icon-sizing (fa2d4f9) onto upstream/main.
Squashed from origin/fix/effort-icon-sizing (fa2d4f9) onto upstream/main.
Moved from kahme247#208, where it landed by mistake.
Squashed from origin/fix/effort-icon-sizing (b447af0) onto upstream/main.
Squashed from origin/fix/effort-icon-sizing (b447af0) onto upstream/main.
Summary
Follow up on the reasoning glyphs introduced in #194.
Verification
npm run typecheck: passed.npm run lint: passed with 11 existing warnings, no errors.npm test: 1,133 passed, 5 platform-specific skips, 0 failures.npm run build: passed (existing static Cache-Control warning).ultraultrainstead ofultra) and passed afterward.No dependencies or API endpoints change. Please squash-merge the iterative sizing commits.