Skip to content

feat(search-query-builder): Add highlighted text#114472

Merged
nsdeschenes merged 7 commits intomasterfrom
nd/feat-search-query-builder-add-highlighted-text
May 4, 2026
Merged

feat(search-query-builder): Add highlighted text#114472
nsdeschenes merged 7 commits intomasterfrom
nd/feat-search-query-builder-add-highlighted-text

Conversation

@nsdeschenes
Copy link
Copy Markdown
Contributor

@nsdeschenes nsdeschenes commented Apr 30, 2026

Small tweak to the search query builder to highlight matching text to make it stand out more.

Example:

Light Mode Dark Mode
Screenshot 2026-05-01 at 12 08 28 Screenshot 2026-05-01 at 12 08 13

Add shared substring highlighting for search query builder suggestions and apply it to filter key matches. Keep fuzzy-only matches unhighlighted so the visual emphasis matches the typed substring.
Apply suggestion substring highlighting to filter value options. Cover both single-value edits and multi-select values so the highlighted match follows the active input text.
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 30, 2026
rbro112 added a commit that referenced this pull request May 1, 2026
Had an issue where #114472 shows
many snapshots removed. Upon further investigation, this was because the
baseSha is being set to the tip of master (`1cfab7e`) and not the true
merge base. This forces the use of the merge-base to avoid issues.
nsdeschenes and others added 2 commits May 1, 2026 12:30
Assert the aggregate suggestion by option name so highlighted label markup does not break the test.

Co-Authored-By: OpenAI Codex <noreply@openai.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 c2411f8. Configure here.

Comment thread static/app/components/searchQueryBuilder/highlightText.tsx
@nsdeschenes nsdeschenes marked this pull request as ready for review May 1, 2026 16:51
@nsdeschenes nsdeschenes requested review from a team as code owners May 1, 2026 16:51
@nsdeschenes nsdeschenes merged commit f13c33a into master May 4, 2026
71 checks passed
@nsdeschenes nsdeschenes deleted the nd/feat-search-query-builder-add-highlighted-text branch May 4, 2026 16:10
cleptric pushed a commit that referenced this pull request May 5, 2026
Had an issue where #114472 shows
many snapshots removed. Upon further investigation, this was because the
baseSha is being set to the tip of master (`1cfab7e`) and not the true
merge base. This forces the use of the merge-base to avoid issues.
cleptric pushed a commit that referenced this pull request May 5, 2026
Small tweak to the search query builder to highlight matching text to
make it stand out more.

---------

Co-authored-by: OpenAI Codex <noreply@openai.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