Reduce mobile toolbar wrapping and relocate fullscreen toggle#114
Merged
Conversation
Mobile toolbar was wrapping to multiple rows on phones. Trim the row and fix an oversized control: - Run/Restart/Stop buttons go icon-only on mobile (the play/stop glyphs are universally recognizable); labels wrapped in xf-hide-mobile. Examples keeps its label (primary new-user action). - Hide the "built ..." version timestamp on mobile. - Fix the About "?" anchor rendering ~40% larger than sibling buttons: it is an <a> (box-sizing: content-box) while the others are <button> (border-box), so the mobile min-width/height:44px rule added padding+border on top. Apply box-sizing: border-box in the toolbar rule so it matches. Also relocate the editor-collapse / game-fullscreen toggle from the toolbar into a single canvas overlay button, pinned top-right (off the top-left where game UI/HUD lives). One button now covers both directions (expand game / restore editor), faint at 0.5 opacity until hovered. This frees another toolbar slot and keeps the affordance on the window it acts on. Shows on desktop too. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
On mobile the top toolbar wraps to multiple rows (Run, Examples, Share, Files, Gist, "?", build timestamp, fullscreen). The exact wrap point varies by device, but we can reduce the likelihood by trimming the row.
Changes
Toolbar space savings (mobile-only, gated to
max-width: 767px/xf-hide-mobile):<a>(browser defaultbox-sizing: content-box) while every other toolbar control is a<button>(border-box), so the mobilemin-width/height: 44pxtouch-target rule added padding+border on top for the anchor — rendering it ~40% larger. Addedbox-sizing: border-boxto the shared toolbar rule so it matches the buttons.Fullscreen toggle relocation (all viewports):
screen-full) / restore editor (screen-normal) — faint at0.5opacity until hovered. Frees another toolbar slot and keeps the affordance on the window it acts on.Notes
dotnet buildpasses clean (0 warnings, 0 errors).🤖 Generated with Claude Code