fix(dashboards): bring selected option to top of visualize dropdown#107355
Merged
nikkikapadia merged 6 commits intomasterfrom Feb 4, 2026
Merged
fix(dashboards): bring selected option to top of visualize dropdown#107355nikkikapadia merged 6 commits intomasterfrom
nikkikapadia merged 6 commits intomasterfrom
Conversation
github-actions
bot
added
the
Scope: Frontend
gggritso
reviewed
Feb 2, 2026
Member
gggritso
left a comment
gggritso
left a comment
There was a problem hiding this comment.
This makes sense, but IMO you should reach out to the Design Engineering team, and ask them if this should be implemented as part of CompactSelect. I think floating the selected values to the top might be better as a component-level behaviour. We do this in multiple places and it'd be good if it was shared
static/app/views/dashboards/widgetBuilder/components/visualize/selectRow.tsx
Outdated
Show resolved
Hide resolved
gggritso
approved these changes
Feb 3, 2026
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
static/app/views/dashboards/widgetBuilder/components/visualize/selectRow.tsx
Show resolved
Hide resolved
jaydgoss
pushed a commit
that referenced
this pull request
Feb 12, 2026
…107355) closes https://linear.app/getsentry/issue/DAIN-389/changes-to-visualize-dropdowns The widget builder visualize dropdowns did not have the selected option in view when re-opening the dropdown. This PR moves the selected option to the top of the dropdown list so that users can see what they have selected in the list. Note: I haven't done this for the metrics dropdown because the dropdown component is reused from the Metrics UI and I didn't want to make changes to that yet.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes https://linear.app/getsentry/issue/DAIN-389/changes-to-visualize-dropdowns
The widget builder visualize dropdowns did not have the selected option in view when re-opening the dropdown. This PR moves the selected option to the top of the dropdown list so that users can see what they have selected in the list.
Note: I haven't done this for the metrics dropdown because the dropdown component is reused from the Metrics UI and I didn't want to make changes to that yet.