Skip to content

fix(ui): Wrap stacktrace link setup button on small screens#107694

Merged
scttcper merged 3 commits intomasterfrom
scttcper/fix-stacktrace-link-responsive-wrap
Feb 5, 2026
Merged

fix(ui): Wrap stacktrace link setup button on small screens#107694
scttcper merged 3 commits intomasterfrom
scttcper/fix-stacktrace-link-responsive-wrap

Conversation

@scttcper
Copy link
Member

@scttcper scttcper commented Feb 5, 2026

before

image

after

image

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 5, 2026
@scttcper scttcper requested a review from a team February 5, 2026 18:51
@scttcper scttcper marked this pull request as ready for review February 5, 2026 18:58
@scttcper scttcper requested a review from a team as a code owner February 5, 2026 18:58
align-items: center;
gap: ${p => p.theme.space.md};
flex-shrink: 0;
margin-left: auto;
Copy link
Member

Choose a reason for hiding this comment

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

Could this be <Flex align="center" justify="end" flexShrink={0} gap="md">?

flex-basis: 100%;
}

> ${FrameActions} {
Copy link
Member

Choose a reason for hiding this comment

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

You could change the props on Flex in response to a prop instead of this CSS rule right?

Copy link
Member Author

Choose a reason for hiding this comment

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

the data is deep in stacktrace link &:has([data-has-setup]) i could add this to FrameActions directly i guess

Copy link
Member

Choose a reason for hiding this comment

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

ah yeah I guess there isn't a good way to do this without a significant refactor

@scttcper scttcper merged commit 83900a3 into master Feb 5, 2026
60 checks passed
@scttcper scttcper deleted the scttcper/fix-stacktrace-link-responsive-wrap branch February 5, 2026 19:33
jaydgoss pushed a commit that referenced this pull request Feb 12, 2026
before

<img width="357" height="633" alt="image"
src="https://github.com/user-attachments/assets/e7d84295-827f-40b2-84f4-f0c8b6233a39"
/>


after

<img width="385" height="370" alt="image"
src="https://github.com/user-attachments/assets/04c02cd0-15a9-4bb1-8096-8c055ab495ad"
/>

---------

Co-authored-by: Claude <noreply@anthropic.com>
dcramer pushed a commit that referenced this pull request Feb 17, 2026
before

<img width="357" height="633" alt="image"
src="https://github.com/user-attachments/assets/e7d84295-827f-40b2-84f4-f0c8b6233a39"
/>


after

<img width="385" height="370" alt="image"
src="https://github.com/user-attachments/assets/04c02cd0-15a9-4bb1-8096-8c055ab495ad"
/>

---------

Co-authored-by: Claude <noreply@anthropic.com>
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