Skip to content

fix(autofix): align project selector and arrow to top in add project modal#116578

Closed
sentry-junior[bot] wants to merge 3 commits into
masterfrom
fix/autofix-modal-left-dropdown-alignment
Closed

fix(autofix): align project selector and arrow to top in add project modal#116578
sentry-junior[bot] wants to merge 3 commits into
masterfrom
fix/autofix-modal-left-dropdown-alignment

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

What

In the "Add Project to Autofix" modal, the left project selector dropdown and the center arrow icon were vertically centered when the right column grew tall (e.g. multiple repo entries). This made the project dropdown appear misaligned — floating in the middle instead of staying at the top.

How

Added alignSelf="start" to the left column's outer Flex wrapper and the center arrow's Flex, so both are pinned to the top of the grid row regardless of how tall the right column (repo entries) grows.

Verified

Code review only — no automated check covers layout alignment.


View Session in Sentry

…modal

When multiple repos are added, the right column grows taller and the
left project selector and center arrow were vertically centered instead
of staying top-aligned. Adding alignSelf='start' pins them to the top
of the grid row.

Co-authored-by: Billy Vong <billy@sentry.io>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 1, 2026
@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ no issues found

@ryan953 ryan953 marked this pull request as ready for review June 1, 2026 16:17
@ryan953 ryan953 requested a review from a team as a code owner June 1, 2026 16:17
@ryan953 ryan953 added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Jun 1, 2026
@billyvg billyvg enabled auto-merge (squash) June 1, 2026 16:35
@billyvg billyvg disabled auto-merge June 1, 2026 18:19
@github-actions github-actions Bot removed the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Jun 1, 2026
@billyvg

billyvg commented Jun 1, 2026

Copy link
Copy Markdown
Member

@ryan953 why are the tests failing 👀

@ryan953

ryan953 commented Jun 1, 2026

Copy link
Copy Markdown
Member

@ryan953 why are the tests failing 👀

@billy same thing here: #116483

i'm thinking it's because there are no related tests to run, and the runner is messed up somehow... hrm

@ryan953

ryan953 commented Jun 1, 2026

Copy link
Copy Markdown
Member

@billyvg I'll go look yak shave it

@ryan953 ryan953 added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Jun 4, 2026
@getsantry

getsantry Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry Bot added the Stale label Jun 26, 2026
@getsantry getsantry Bot closed this Jul 4, 2026
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 Stale Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants