Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable autocompletion for website suggestions #3351

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

tomasstrba
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/0/1208219569168394/f

Description:
Autocompletion (automatic selection of the first Top Hit suggestion) disabled for website suggestions.

Steps to test this PR:

  1. Clean the application or make sure you have a testing URL which is not stored in history. I'll use bbc.com in this example
  2. Type bb into the address bar
  3. Make sure bbc.com website suggestion is the first suggestion (or other URL alternatives), but it's NOT automatically selected
  4. Navigate to bbc.com (exactly) and wait until the website loads
  5. Open a new tab, type bb into the address bar again
  6. Make sure bbc.com is a first suggestion (based on history) and autocompleted (automatically selected)

Definition of Done:


Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

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

LGTM

Couldn't get it working at first, because for some reason it wasn't picking up the commit with the change in. I switched to a commit ref and used the latest one and it then started working. Better to use explicit commit refs tbh. Hopefully this BSK dance will be gone soon 🙏

Copy link
Contributor

github-actions bot commented Oct 4, 2024

This PR has been inactive for more than 7 days and will be automatically closed 7 days from now.

@github-actions github-actions bot added the stale label Oct 4, 2024
tomasstrba added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Oct 10, 2024
Task/Issue URL: https://app.asana.com/0/414235014887631/1208219569168394/f
iOS PR: duckduckgo/iOS#3401
macOS PR: duckduckgo/macos-browser#3351
What kind of version bump will this require?: Major/Minor/Patch

**Description**:

Disable autocompletion for website suggestions
@tomasstrba tomasstrba removed the stale label Oct 10, 2024
@tomasstrba tomasstrba merged commit 2bfebbb into main Oct 10, 2024
19 checks passed
@tomasstrba tomasstrba deleted the tom/disable-domain-suggestions branch October 10, 2024 12:20
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