fix(dashboards): Hide legend alias option for details widget type#112957
fix(dashboards): Hide legend alias option for details widget type#112957DominikB2014 merged 2 commits intomasterfrom
Conversation
|
🚨 Warning: This pull request contains Frontend and Backend changes! It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently. Have questions? Please ask in the |
Sentry Snapshot Testing
|
The details widget displays data as key-value pairs with a single query, so a legend alias serves no purpose. Add it to the exclusion list alongside tables, big numbers, and categorical bars. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
a9f8c5d to
b358c11
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit b358c11. Configure here.
The details widget only supports a single query (the state reducer already slices to one on switch), but the "+ Add Filter" button was still rendering. Add DETAILS to the canAddSearchConditions exclusion list, consistent with TABLE and BIG_NUMBER. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Hide the legend alias input in the query filter builder when the details
widget type is selected. The details widget displays data as key-value
pairs with a single query, so a legend alias serves no purpose — similar
to tables and big numbers which are already excluded.
Adds a test confirming the alias input is not rendered for details widgets.