Skip to content

Bugfix: Query suggestions only work after more than 3 characters are typed and a character is deleted - #73

Merged
Garneauma merged 1 commit into
ServiceCanada:mainfrom
cfoss-coveo:bugfix/447196-query-suggestions-mobile-only-not-working
Aug 11, 2026
Merged

Bugfix: Query suggestions only work after more than 3 characters are typed and a character is deleted#73
Garneauma merged 1 commit into
ServiceCanada:mainfrom
cfoss-coveo:bugfix/447196-query-suggestions-mobile-only-not-working

Conversation

@cfoss-coveo

Copy link
Copy Markdown
Contributor

Description

  • Fix for Query Suggestions on Android.
  • Uses Input events rather that Keyboard events to better detect typing.
  • Fixes for both the "top right" search QS and the main search QS

Details

  • Input events more reliable across devices
  • Tightened up logic around "enter" key

Azure DevOps Work Item(s)

  • Defect 447196: Query Suggestions - MOBILE ONLY Query suggestions only work after more than 3 characters are typed and a character is deleted

Test Cases

Pre-requisite

  • Deploy the search UI to a URL you can browse in mobile. If using Android Studio to test, you can access localhost at 10.0.2.2

Top right search

  1. Load the top right search in mobile (Android)
  2. Type 3 letters into the search. For example, can
    • Query suggestion should appear
  3. Backspace
    • Query suggestion should disappear
  4. Type a different letter. i.e., r to make the query car
    • Query suggestion should appear again and be different

Basic search

  1. Load the basic search UI in mobile (Android)
  2. Repeat the same process as above

- Input events more reliable across devices
- Tightened up logic around "enter" key
@cfoss-coveo
cfoss-coveo requested a review from a team August 10, 2026 23:22
@cfoss-coveo

Copy link
Copy Markdown
Contributor Author

@Garneauma Garneauma 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.

@cfoss-coveo Everything works as expected, on mobile and desktop. OnInput event is indeed better than OnKeyup. Thank you!

@Garneauma

Copy link
Copy Markdown
Contributor

@ServiceCanada/sc-pull-request-approvers For your review.

@Yves-Denis Yves-Denis left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Security Assessment Completed - Passed
No security vulnerabilities introduced with changes to .js files.

@Garneauma
Garneauma merged commit cb31c44 into ServiceCanada:main Aug 11, 2026
1 check 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.

3 participants