Skip to content

Conversation

JonasBa
Copy link
Member

@JonasBa JonasBa commented Oct 10, 2025

Fix alert background so that it won't bleed into the page

CleanShot.2025-10-10.at.14.00.50.mp4

@JonasBa JonasBa requested a review from a team as a code owner October 10, 2025 21:02
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 10, 2025
@JonasBa JonasBa force-pushed the jb/alert/primary-background branch from 5dd13e3 to 4ade164 Compare October 10, 2025 21:02
cursor[bot]

This comment was marked as outdated.

`;
}
return css`
background: ${tokens.background};
Copy link
Member Author

Choose a reason for hiding this comment

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

@cursoragent apply the same linear gradient here as on line 119

Copy link
Contributor

@cursor cursor bot Oct 10, 2025

Choose a reason for hiding this comment

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

I modified the alert component's background styling to enhance its visual presentation and ensure a consistent appearance across different themes. This change specifically updates how alert backgrounds are rendered, moving to a more robust gradient-based approach.

  • I updated the generateAlertBackground function within static/app/components/core/alert/alert.chonk.tsx.
  • I changed the background property to background-image, applying a layered linear-gradient.
  • This new background-image now uses linear-gradient(${tokens.background}) on top of linear-gradient(${theme.colors.background.primary}), providing a more sophisticated background and ensuring a solid fallback color.

Open in Cursor Open in Web

Learn more about Cursor Agents

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.

3 participants