Skip to content

Fix/mobile responsiveness#1411

Closed
scytherswings wants to merge 6 commits intooraios:mainfrom
scytherswings:fix/mobile-responsiveness
Closed

Fix/mobile responsiveness#1411
scytherswings wants to merge 6 commits intooraios:mainfrom
scytherswings:fix/mobile-responsiveness

Conversation

@scytherswings
Copy link
Copy Markdown
Contributor

Fix: Dashboard Mobile & Tablet Responsiveness

Problem

The Serena dashboard was completely broken on any screen narrower than 1280px. A single CSS rule — min-width: 1280px on the
#frame container — forced horizontal scrolling at all smaller widths and silently disabled every responsive media query in the
stylesheet. Additionally, the platinum banner was embedded inside the header's flex layout, causing cascading sizing and
overlap issues at mid-range viewports.

Changes

Root cause fix

  • Removed min-width: 1280px from #frame, enabling all responsive breakpoints to actually fire

Header layout

  • Header collapses to a compact column layout at ≤768px with height: auto so it sizes naturally
  • Logo scales down to 100px tall on mobile (was inheriting the full 150px desktop value)
  • Removed a pre-existing body { padding-top: 140px } that assumed a fixed/sticky header (the header is neither — it was
    creating a large blank space at the top of the page on mobile)
  • header-nav and header-left set to height: auto at mobile so they don't try to fill the desktop height

Breakpoints

  • 1024px: overview grid collapses to single column; stat tool name and config grid label column widths reduced for tablet
  • 768px: full mobile column layout, compact logo, log container height recalculated
  • 480px (new): tighter padding for small phones

Platinum banner

  • Moved out of the header entirely into its own .platinum-banners-row div between the header and main content
  • Capped at max-height: 156px with overflow: hidden so it can't blow out on large screens
  • Images get max-width: 100% to prevent async-loaded content from overflowing the container

Gold banners ("Support Serena")

  • Restored to original location in overview-right — no functional changes

Testing

Open the dashboard and use browser DevTools responsive mode at:

  • 1024px — overview collapses to single column, no horizontal scroll
  • 768px — header stacks vertically, compact logo, banner below header
  • 480px — padding tightens, no overflow
  • 375px — no horizontal scrollbar at any point

BEFORE:
Screenshot 2026-04-24 at 1 11 18 PM

AFTER:
Screenshot 2026-04-24 at 1 11 27 PM

For context, I have no interest in spending more time redesigning the UX, I would recommend considering a redesign of the CTA for support and making a carousel or two column layout underneath the navbar for your announcements/notifications/banners. Right now it feels awkward on desktop but I don't have have more time to spend.

scytherswings and others added 5 commits April 24, 2026 13:16
- Remove min-width: 1280px constraint from #frame to enable media queries
- Extend 1024px media query with tablet-optimized sizing for stat tools and config grid
- Add log container height adjustment for mobile views in 768px media query
- Add 480px breakpoint for small phone screens with adjusted padding and sizing

This fixes the dashboard to properly collapse and adapt layout on tablet (1024px)
and mobile (768px and 480px) viewports without horizontal scrolling.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Remove pre-existing body padding-top: 140px that assumed a fixed header
  (header is not fixed/sticky, so this created a large blank space at top)
- Set header height: auto at 768px so it collapses naturally in column layout
- Constrain logo image to 55px height at mobile instead of inheriting 150px
- Set logo-container and header-nav to height: auto at mobile
- Add header-left centering at mobile for balanced appearance
- Constrain header-banner to max-height: 55px at mobile

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Extract #platinum-banners from header-left into its own .platinum-banners-row
  div between the header and main content
- Cap height at 150px with overflow: hidden so it can't expand unchecked
- Add max-width: 100% to logo image and platinum banner images to prevent overflow
- Restore gold-banners-section and .gold-banner CSS to original state
- Remove dead .header-banner rules from 768px media query

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@scytherswings scytherswings force-pushed the fix/mobile-responsiveness branch from a6f5bd9 to 4feaf46 Compare April 24, 2026 17:16
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@opcode81
Copy link
Copy Markdown
Contributor

Thanks for your efforts, but on on desktop, the platinum banner now renders below the header section even though there is enough space. This unnecessarily wastes vertical space.

image

@opcode81
Copy link
Copy Markdown
Contributor

@scytherswings so you don't want to improve upon your solution?

@scytherswings
Copy link
Copy Markdown
Contributor Author

@opcode81 I probably should have opened this as an issue not a PR since it involves design. I burned an hour of my time iterating through various approaches already this morning to land back at this simple tweak, if this is something that ya'll are interested in please carry forward, but I don't have any more time to spend on UX, sorry, I'm a backend dev mostly.

See my original suggestions:

For context, I have no interest in spending more time redesigning the UX, I would recommend considering a redesign of the CTA for support and making a carousel or two column layout underneath the navbar for your announcements/notifications/banners. Right now it feels awkward on desktop but I don't have have more time to spend.

@opcode81
Copy link
Copy Markdown
Contributor

opcode81 commented Apr 24, 2026

Makes sense. Personally, I won't be exploring this further, because lower resolutions are unlikely to be relevant in the majority of usage scenarios. After all, developers are typically working in desktop environments with at least 1080p. Hence my decision to limit it to 1280 for the sake of simplicity.

@scytherswings
Copy link
Copy Markdown
Contributor Author

I work on a macbook pro and had serena's dashboard side-by-side with my terminal while trying to troubleshoot it and get it set up, thus this work, but I get it.

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