Skip to content

Replace search filters with chips and a labeled search field - #113

Merged
thcathy merged 4 commits into
masterfrom
cursor/search-dictation-filter-chips
Oct 5, 2026
Merged

thcathy merged 4 commits into
masterfrom
cursor/search-dictation-filter-chips

Conversation

@thcathy

@thcathy thcathy commented Oct 4, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • The keyword field is a Search input with the hint “Title, words, or ID”. Enter runs the same search as the button when the form is valid, and does nothing when it is not.
  • Suitable, Type, and Date are one wrapping row of chips. Each chip shows its title and the current value, and opens a popover. Choosing a value does not search.
  • Created By is always visible, above the chips. More Options is removed.
  • The dictation list preview is the last line of each row.

Search request mapping is unchanged: type Any is sent as null, Suitable Any is still sent as Any, and a date of Any omits minDate.

Tests

  • npx ng test --browsers=ChromeHeadlessCI --watch=false --include='**/search-dictation.page.spec.ts' --include='**/dictation-list.spec.ts' — 16 passed

Test evidence

Local app at http://localhost:4200/search-dictation. Each clip opens the page, chooses Sentence and Senior Primary from the chips, shows the 3-character error for ab, then searches apple. iOS and Android are the same local app in Safari and Chrome at phone size. No native simulator was running.

Web

web-desktop.mp4

iPhone (Safari)

ios-safari.mp4

Android (Chrome)

android-chrome.mp4

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

thcathy and others added 4 commits October 4, 2026 21:42
Enter runs the same search as the button when the form is valid. Suitable, type, and date stay visible as chips that open a popover, and Created By sits above them. The dictation list preview is the last line of each row.

Co-authored-by: Cursor <cursoragent@cursor.com>
SonarCloud treats the unhandled search() promise as a reliability bug.

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

sonarqubecloud Bot commented Oct 5, 2026

Copy link
Copy Markdown

@thcathy
thcathy merged commit 590a9d5 into master Oct 5, 2026
5 checks passed
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.

1 participant