Skip to content

fix(dashboards): Cap global filter trigger width to prevent layout shifts#111238

Merged
gggritso merged 2 commits intomasterfrom
georgegritsouk/dain-1377-layout-shifts-when-long-filter-name-used
Mar 23, 2026
Merged

fix(dashboards): Cap global filter trigger width to prevent layout shifts#111238
gggritso merged 2 commits intomasterfrom
georgegritsouk/dain-1377-layout-shifts-when-long-filter-name-used

Conversation

@gggritso
Copy link
Copy Markdown
Member

@gggritso gggritso commented Mar 20, 2026

Wraps each filter selector trigger in a Container with a maxWidth of 300px so that selecting a long filter value doesn't cause the trigger button to expand and push other elements around in the filters bar as much. It still will a bit!

Fixes DAIN-1377

…ifts

Wrap the OverlayTrigger.Button in each filter selector with a Container
capped at 300px max-width. This prevents the trigger from expanding
when a long filter value is selected, eliminating the layout shift in
the filters bar.

A shared FILTER_SELECTOR_MAX_WIDTH constant is exported from
filterSelector.tsx and imported by numericFilterSelector.tsx so the
cap stays in sync across both selector types.

Fixes DAIN-1377
Co-Authored-By: Claude <noreply@anthropic.com>
@gggritso gggritso requested a review from a team as a code owner March 20, 2026 19:03
@linear-code
Copy link
Copy Markdown

linear-code bot commented Mar 20, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 20, 2026
@gggritso gggritso merged commit e0ed813 into master Mar 23, 2026
69 checks passed
@gggritso gggritso deleted the georgegritsouk/dain-1377-layout-shifts-when-long-filter-name-used branch March 23, 2026 19:26
@github-actions github-actions bot locked and limited conversation to collaborators Apr 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants