Skip to content

Show a clearer no-results state on dictation search - #114

Merged
thcathy merged 4 commits into
masterfrom
cursor/search-dictation-no-results-18da
Oct 7, 2026
Merged

thcathy merged 4 commits into
masterfrom
cursor/search-dictation-no-results-18da

Conversation

@thcathy

@thcathy thcathy commented Oct 7, 2026 •

Copy link
Copy Markdown
Owner

Summary

When Search Dictation returns zero hits, the empty card says No dictation matched. and offers Clear. It does not repeat the keyword or list Suitable, Type, or date. The empty state lives on /search-dictation only.

Clear resets the keyword, Created By, and the three chips, and removes the empty card. It does not run a search.

How to verify on /search-dictation

  • Search a keyword that should miss, such as zzzzqqqxxx, with or without Suitable, Type, or date set. Expect No dictation matched. and Clear.
  • Clear: keyword and filters reset, and the empty card disappears.
  • A search that has hits still shows the result list.

Tests

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

19 specs passed.

no-dictation-matched.mp4
Empty search says No dictation matched

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

cursoragent and others added 4 commits October 7, 2026 10:31
Name the keyword and any Suitable, type, or date filters, and let
learners loosen those filters or clear the search in one step.

Co-authored-by: Timmy Wong <thcathy@gmail.com>
Share one filter reset for Soften filters and Clear, and leave the
chips alone when the form is invalid.

Co-authored-by: Timmy Wong <thcathy@gmail.com>
Keep the zero-hit message and Clear. Drop the button, its copy, and
the shared reset that only Clear still used.

Co-authored-by: Timmy Wong <thcathy@gmail.com>
Drop the keyword and filter line from the empty search card.
Clear still resets the form.

Co-authored-by: Timmy Wong <thcathy@gmail.com>
@sonarqubecloud

sonarqubecloud Bot commented Oct 7, 2026

Copy link
Copy Markdown

@thcathy
thcathy marked this pull request as ready for review October 7, 2026 14:05
@thcathy
thcathy merged commit 4a4c7d9 into master Oct 7, 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.

2 participants