Skip to content

Web dashboard: stretch layout to full screen width#590

Open
RyuShai wants to merge 1 commit into
AlexsJones:mainfrom
RyuShai:fix/dashboard-full-width
Open

Web dashboard: stretch layout to full screen width#590
RyuShai wants to merge 1 commit into
AlexsJones:mainfrom
RyuShai:fix/dashboard-full-width

Conversation

@RyuShai

@RyuShai RyuShai commented May 31, 2026

Copy link
Copy Markdown

Summary

The llmfit serve web dashboard .page-shell container was hard-capped at max-width: 1640px and centered with margin: 0 auto. On wide / ultrawide displays (e.g. 21:9) this left large empty margins on both sides and the model table did not use the available horizontal space.

  • Removed the fixed cap → layout now fills 100% of the viewport width.
  • Added responsive horizontal padding clamp(1.1rem, 2.5vw, 3rem) so content keeps comfortable gutters on very wide screens.

Single-line CSS change in llmfit-web/src/styles.css (.page-shell).

Testing

  • npm test → all passing.
  • npm run build succeeds; verified the rebuilt binary serves the updated bundle via llmfit serve.

🤖 Generated with Claude Code

The dashboard `.page-shell` container was hard-capped at `max-width: 1640px`
and centered with `margin: 0 auto`. On wide / ultrawide displays (e.g. 21:9)
this left large empty margins on both sides and the model table did not use
the available horizontal space.

Remove the fixed cap so the layout fills 100% of the viewport width, and add
responsive horizontal padding via `clamp(1.1rem, 2.5vw, 3rem)` so content
still keeps comfortable gutters on very wide screens.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@RyuShai RyuShai force-pushed the fix/dashboard-full-width branch from f687388 to 78514c9 Compare May 31, 2026 11:39
@RyuShai RyuShai changed the title Web dashboard: full-width layout + per-column table filters Web dashboard: stretch layout to full screen width May 31, 2026
@AlexsJones

Copy link
Copy Markdown
Owner

Thanks @RyuShai, and apologies for the slow first look. The clamp() gutter padding is a nice touch. One thing worth deciding before this lands: dropping the cap to max-width: 100% makes the model table span the full width of a 21:9/32:9 display, and very long line-lengths in a data table can actually hurt scan-ability — the original 1640px cap was likely deliberate for that reason.

A middle ground that fixes the "big empty margins" complaint without going fully edge-to-edge would be to raise the cap rather than remove it, e.g. max-width: min(2200px, 100%), keeping your clamp() padding. That reclaims most of the wasted space on ultrawides while still bounding line length on the extreme end.

Either way it's a visual-direction call, so I've left the final say to Alex. Happy to re-review quickly if you adjust.

🤝 Handled by Alex's repo steward — replies reviewed by @AlexsJones

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