Skip to content

Conversation

@MichaelSun48
Copy link
Contributor

@MichaelSun48 MichaelSun48 commented Jun 5, 2025

fixes #92151

Fixes an issue where the event attachment name would overflow off the card and obscure the ellipsis menu. Also cleans up some styles.

Before:

Screenshot 2025-06-05 at 1 13 32 PM

After:

Screenshot 2025-06-05 at 1 14 02 PM

@MichaelSun48 MichaelSun48 requested a review from a team as a code owner June 5, 2025 20:14
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 5, 2025
const AttachmentName = styled('span')`
font-weight: ${p => p.theme.fontWeightBold};
display: flex;
${p => p.theme.overflowEllipsis};
Copy link
Member

Choose a reason for hiding this comment

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

wait doens't overflowEllipsis give us some of these styles too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah good point, the styles under this are redundant. removed

@MichaelSun48 MichaelSun48 enabled auto-merge (squash) June 6, 2025 17:13
@MichaelSun48 MichaelSun48 merged commit 5ec3024 into master Jun 6, 2025
42 checks passed
@MichaelSun48 MichaelSun48 deleted the msun/fixAttachmentOverflowIssue branch June 6, 2025 17:15
andrewshie-sentry pushed a commit that referenced this pull request Jun 19, 2025
…card (#92990)

fixes #92151

Fixes an issue where the event attachment name would overflow off the
card and obscure the ellipsis menu. Also cleans up some styles.

Before: 

![Screenshot 2025-06-05 at 1 13
32 PM](https://github.com/user-attachments/assets/d3288759-faa2-4dc6-a247-232de1a55175)

After: 

![Screenshot 2025-06-05 at 1 14
02 PM](https://github.com/user-attachments/assets/28b046c0-5e52-4bc5-9fcd-3ec6a962824d)
@github-actions github-actions bot locked and limited conversation to collaborators Jun 22, 2025
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.

Screenshot preview not truncating file name which overflows

3 participants