Skip to content

ref(pagefilters) align checkbox#108338

Merged
JonasBa merged 1 commit intomasterfrom
jb/pagefilters/checkbox-align
Feb 17, 2026
Merged

ref(pagefilters) align checkbox#108338
JonasBa merged 1 commit intomasterfrom
jb/pagefilters/checkbox-align

Conversation

@JonasBa
Copy link
Member

@JonasBa JonasBa commented Feb 17, 2026

Best effort to align pagefilters checkbox. Most visible on the env page filter

before
CleanShot 2026-02-17 at 04 01 50@2x
after
CleanShot 2026-02-17 at 04 01 59@2x

@JonasBa JonasBa requested a review from a team as a code owner February 17, 2026 12:00
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 17, 2026
display: flex;
gap: ${p => p.theme.space.md};
justify-content: space-between;
align-items: center;
Copy link
Member Author

Choose a reason for hiding this comment

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

This ensures that the entire row is centered in the container

aria-label={t('Select %s', project.slug)}
tabIndex={-1}
/>
<Flex align="center" gap="sm" flex="1 1 100%">
Copy link
Member Author

Choose a reason for hiding this comment

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

Grow the Container to full height so that we can center the checkbox

gap="md"
align="center"
flow="column"
height="100%"
Copy link
Member Author

Choose a reason for hiding this comment

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

Stretch the grid container over the entire height of the row

@JonasBa JonasBa merged commit 15546ac into master Feb 17, 2026
64 checks passed
@JonasBa JonasBa deleted the jb/pagefilters/checkbox-align branch February 17, 2026 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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