Skip to content

Improve mobile toolbar clarity - #56

Merged
a-baran-orhan merged 1 commit into
mainfrom
codex/mobile-toolbar-clarity
Jul 14, 2026
Merged

Improve mobile toolbar clarity#56
a-baran-orhan merged 1 commit into
mainfrom
codex/mobile-toolbar-clarity

Conversation

@a-baran-orhan

@a-baran-orhan a-baran-orhan commented Jul 14, 2026

Copy link
Copy Markdown
Member

Summary

  • align all mobile toolbar controls to a consistent 42px height
  • remove the empty grid gap and pair a wide Copy link action with a compact GitHub issue shortcut
  • show a clear mobile “Copy link” label while retaining compact desktop copy
  • add accessible pending, success, and error feedback for clipboard actions

Why

The mobile toolbar inherited centered grid alignment, which left controls in the same row at different heights and produced an unused grid cell. The share confirmation was also written into a label hidden by the mobile stylesheet, so users could not tell whether copying succeeded.

Impact

Mobile users get a tighter, aligned toolbar with clearer action priority and immediate “Copying…” feedback followed by a high-contrast success or error state. Feedback now opens GitHub’s issue composer instead of email, while desktop remains compact to avoid toolbar overflow.

Validation

  • npm run build
  • git diff --check
  • visually checked at a 390px mobile viewport
  • checked the 320px edge case with no horizontal overflow
  • verified consistent 42px control heights and the pending copy state

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
posecode Ready Ready Preview, Comment Jul 14, 2026 3:17pm

@a-baran-orhan
a-baran-orhan marked this pull request as ready for review July 14, 2026 15:17
@a-baran-orhan
a-baran-orhan merged commit 2b21d10 into main Jul 14, 2026
3 checks passed
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