Skip to content

[CONCEPT] feat(app-ui): measure responsive props on the page column - #885

Closed
exromany wants to merge 1 commit into
mainfrom
feat/app-ui-page-column
Closed

exromany wants to merge 1 commit into
mainfrom
feat/app-ui-page-column

Conversation

@exromany

@exromany exromany commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Stack, Grid and GridItem breakpoint keys now measure the page column instead of the viewport when they sit inside SidebarInset. Layouts no longer gain columns when the sidebar collapses.

  • Before: viewport keys gave 4 columns at 1280 (with the sidebar) and 3 at 1279 (sidebar closed, content 300px wider).
  • Now: with the sidebar shown, content keys see the viewport minus 296px. Same scale and keys, no new numbers.
  • Outside a page container, e.g. a portaled Modal or an app without SidebarInset, behaviour is unchanged. The mixin emits the existing @media rules plus @container lido-app-ui-page rules after them. The column is never wider than the viewport, so the narrower result wins.
  • What to watch for: sidebar layouts at ≥1280 now resolve one key narrower (1280–1575 → lg/ml). Check lgDown/mlDown values in those apps.
  • Hidden follows the column too. below hides when the column is narrower. above hides on the viewport but shows again when the column is narrower. New media-only sidebar key (1280, the drawer switch) for shell chrome. The header logo in the recipes now uses Hidden above='sidebar', which also fixes the missing logo at 1024–1279. New HiddenBreakpoint type export.
  • Not changed: Modal, Logo, the sidebar and page gutters stay viewport-based.
  • New pageContainerName export: apps without SidebarInset can make their main column the page container.
  • Also fixes stale sm px in the Grid and Stack story docs.
1280, sidebar shown (column 984) 1279, drawer (column 1279)

@exromany
exromany requested a review from a team as a code owner October 30, 2026 10:45
@exromany exromany changed the title feat(app-ui): measure responsive props on the page column [CONCEPT] feat(app-ui): measure responsive props on the page column Oct 30, 2026
Stack, Grid and GridItem breakpoint keys now also respond to a
`lido-app-ui-page` container, set on SidebarInset. With the sidebar
shown, content sees the viewport minus the sidebar, so layouts no longer
gain columns when the sidebar collapses at 1280px. Outside a page
container the media rules still apply unchanged. Exports
pageContainerName for apps without SidebarInset.
@exromany
exromany force-pushed the feat/app-ui-breakpoints branch from 391c8ec to 95faa9f Compare October 30, 2026 12:39
@exromany
exromany force-pushed the feat/app-ui-page-column branch from 3e7d95b to b721a73 Compare October 30, 2026 12:39
Base automatically changed from feat/app-ui-breakpoints to main October 30, 2026 14:42
@preview-stands

preview-stands Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Preview stand status

Stand was demolished

@exromany exromany closed this Oct 30, 2026
@exromany
exromany deleted the feat/app-ui-page-column branch October 30, 2026 18:18
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