Skip to content

fix(ui): Rank trace group by matches with fzf#115032

Merged
scttcper merged 1 commit into
masterfrom
scttcper/group-by-search
May 8, 2026
Merged

fix(ui): Rank trace group by matches with fzf#115032
scttcper merged 1 commit into
masterfrom
scttcper/group-by-search

Conversation

@scttcper
Copy link
Copy Markdown
Member

@scttcper scttcper commented May 6, 2026

Follow-up to #109957. Keep the known-field search behavior there, but use fzf to rank matched Group By options so searching org puts organization.slug before flag.evaluation.feature.organizations.

before

image

after

image

The Explore trace Group By dropdown bucketed all partial known fields together, so matching org could keep flag.evaluation.feature.organizations ahead of organization.slug just because of the base option order.

Keep the substring filter behavior, but use fzf to rank matches inside that filtered set. Adds a local prefix bump so the obvious org prefix wins without changing global CompactSelect search.
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.44%

@scttcper scttcper marked this pull request as ready for review May 6, 2026 22:27
@scttcper scttcper requested a review from a team as a code owner May 6, 2026 22:27
@scttcper scttcper merged commit a9b03c7 into master May 8, 2026
76 checks passed
@scttcper scttcper deleted the scttcper/group-by-search branch May 8, 2026 15:51
nsdeschenes added a commit that referenced this pull request May 11, 2026
Continue the work from #115032, and extend it to the visualize dropdown,
and column editors.

---------

Co-authored-by: OpenAI Codex <noreply@openai.com>
dashed pushed a commit that referenced this pull request May 11, 2026
Continue the work from #115032, and extend it to the visualize dropdown,
and column editors.

---------

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