Skip to content

ENG-1840: Align advanced search highlight with Roam/BlueprintJS#1117

Open
trangdoan982 wants to merge 3 commits into
mainfrom
eng-1840-the-purple-highlight-doesnt-match
Open

ENG-1840: Align advanced search highlight with Roam/BlueprintJS#1117
trangdoan982 wants to merge 3 commits into
mainfrom
eng-1840-the-purple-highlight-doesnt-match

Conversation

@trangdoan982

@trangdoan982 trangdoan982 commented Jun 9, 2026

Copy link
Copy Markdown
Member
image

Summary

  • Replace the purple custom color with a gray that matches Roam
  • can't use bp3.menu-item-active because the default is blue instead of gray

Test plan

  • Open Advanced Node Search in Roam and search for a keyword
  • Confirm focused result rows use gray Blueprint hover styling (not purple or blue)
  • Verify ArrowUp/ArrowDown navigation and scroll-into-view for long result lists
  • Confirm filter and sort buttons show Blueprint active state when open or applied
  • Verify Enter, Shift+Enter, Cmd/Ctrl+Enter, and Escape still work as expected

Made with Cursor


Open in Devin Review

Replace custom purple selection chrome with Blueprint menu-item hover gray for result rows and native active states for filter/sort toolbar buttons.

Co-authored-by: Cursor <cursoragent@cursor.com>
@linear-code

linear-code Bot commented Jun 9, 2026

Copy link
Copy Markdown

ENG-1840

@supabase

supabase Bot commented Jun 9, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@vercel

vercel Bot commented Jun 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
discourse-graph Skipped Skipped Jun 9, 2026 9:04pm

Request Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 2 potential issues.

View 1 additional finding in Devin Review.

Open in Devin Review

Comment thread apps/roam/src/components/AdvancedNodeSearchDialog/AdvancedSearchDialog.tsx Outdated
Comment thread apps/roam/src/components/DiscourseNodeSortControl.tsx Outdated
Remove unnecessary querySelector type assertion and unused useMemo import.

Co-authored-by: Cursor <cursoragent@cursor.com>
Comment thread apps/roam/src/components/AdvancedNodeSearchDialog/AdvancedSearchDialog.tsx Outdated

@mdroidian mdroidian left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there no way to do this with built in params/tailwind colors? Seems odd that we have to code in the exact rgba for on hover.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants