Skip to content

Conversation

@leeandher
Copy link
Member

Before:
image

After:
image

Before UI2:
image

After UI2:
image

@leeandher leeandher requested a review from a team as a code owner July 11, 2025 16:53
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 11, 2025
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Bug: Chart Series Color Change Affects Visibility

The user series color in the chart now uses translucentGray300 when isUnfilteredStatsEnabled is false. This unintended change makes the series very light and less prominent in unfiltered views, where it was previously always purple200.

static/app/views/issueDetails/streamline/eventGraph.tsx#L379-L380

borderColor: theme.translucentGray200,
color: isUnfilteredStatsEnabled ? theme.purple300 : translucentGray300,

Fix in CursorFix in Web


Was this report helpful? Give feedback by reacting with 👍 or 👎

@leeandher
Copy link
Member Author

leeandher commented Jul 11, 2025

Bug: Chart Series Color Change Affects Visibility

The user series color in the chart now uses translucentGray300 when isUnfilteredStatsEnabled is false. This unintended change makes the series very light and less prominent in unfiltered views, where it was previously always purple200.

static/app/views/issueDetails/streamline/eventGraph.tsx#L379-L380

Fix in CursorFix in Web

Was this report helpful? Give feedback by reacting with 👍 or 👎

Since the two series never show at the same time, It's okay they share the same base colour of gray. The 'matching x' colouring behaviour is what I was trying to make consistent

@leeandher leeandher merged commit fd77593 into master Jul 15, 2025
55 of 57 checks passed
@leeandher leeandher deleted the leander/graph-colors branch July 15, 2025 14:05
@github-actions github-actions bot locked and limited conversation to collaborators Jul 31, 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.

3 participants