Skip to content

fix(ui): prevent wide branding banner from pushing toolbar off screen on mobile#1177

Merged
perber merged 1 commit into
mainfrom
fix/branding-banner-width-mobile
Jun 19, 2026
Merged

fix(ui): prevent wide branding banner from pushing toolbar off screen on mobile#1177
perber merged 1 commit into
mainfrom
fix/branding-banner-width-mobile

Conversation

@perber

@perber perber commented Jun 19, 2026

Copy link
Copy Markdown
Owner

Allow the logo container to shrink (min-w-0 + shrink), reduce its margins on small screens, and cap the image width at 96px on mobile so the save button and toolbar remain visible in portrait orientation.

… on mobile

Allow the logo container to shrink (min-w-0 + shrink), reduce its
margins on small screens, and cap the image width at 96px on mobile
so the save button and toolbar remain visible in portrait orientation.
Copilot AI review requested due to automatic review settings June 19, 2026 12:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the UI header layout styling to ensure an oversized branding/logo image can’t push the editor toolbar (including the save button) off-screen on small/mobile viewports.

Changes:

  • Allow the header logo/title flex item to shrink by adding min-w-0 and shrink, and reduce its horizontal margins on small screens.
  • Cap the logo image max width to 96px on small screens (max-sm:max-w-24) to prevent overly wide branding images from consuming the header.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@perber
perber merged commit 37b544a into main Jun 19, 2026
10 checks passed
@perber
perber deleted the fix/branding-banner-width-mobile branch June 19, 2026 13:06
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.

2 participants