Skip to content

ref(issue-list): Simplify monitors dropdown labels and item behavior#112450

Merged
JonasBa merged 3 commits intomasterfrom
jb/ref/monitor-dropdown
Apr 9, 2026
Merged

ref(issue-list): Simplify monitors dropdown labels and item behavior#112450
JonasBa merged 3 commits intomasterfrom
jb/ref/monitor-dropdown

Conversation

@JonasBa
Copy link
Copy Markdown
Member

@JonasBa JonasBa commented Apr 8, 2026

Simplify the monitors dropdown in the issue list by updating item labels to a cleaner "View Cron/Uptime Monitors (N)" format indicating the link behavior.

I initially thought that the arrows were indicating sub-menus, which was quite confusing given that they are links...

Note for the design system here that we should probably not allow trailing items in dropdown menus - those should be reserved for the chevron sub-menu indicator and consumers should only use leading icons to indicate entries

Replace verbose "N Active Cron/Uptime Monitors" labels with cleaner
"View Cron/Uptime Monitors (N)" format. Remove trailing arrow icons
and disabled state since the dropdown handles item styling natively.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 8, 2026
@JonasBa JonasBa marked this pull request as ready for review April 8, 2026 05:58
@JonasBa JonasBa requested a review from a team as a code owner April 8, 2026 05:58
@JonasBa JonasBa requested a review from a team April 8, 2026 05:59
Update test assertions to match the refactored label format from
'Active Cron Monitors' to 'View Cron Monitors' following the recent
monitors dropdown label simplification.

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Copy Markdown
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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit bfa8a5e. Configure here.

Comment thread static/app/views/issueList/monitorsDropdown.tsx Outdated
Update cron and uptime monitor dropdown labels to explicitly say
'View Active Cron Monitors' and 'View Active Uptime Monitors' to
clarify that the count reflects active monitors only.

Co-Authored-By: Claude <noreply@anthropic.com>
@JonasBa JonasBa merged commit 1cf1f08 into master Apr 9, 2026
66 checks passed
@JonasBa JonasBa deleted the jb/ref/monitor-dropdown branch April 9, 2026 02:21
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
…112450)

Simplify the monitors dropdown in the issue list by updating item labels
to a cleaner "View Cron/Uptime Monitors (N)" format indicating the link
behavior.

I initially thought that the arrows were indicating sub-menus, which was
quite confusing given that they are links...

Note for the design system here that we should probably not allow
trailing items in dropdown menus - those should be reserved for the
chevron sub-menu indicator and consumers should only use leading icons
to indicate entries

---------

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