Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Conversation

@SabrinaTardio
Copy link
Collaborator

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

Description: at the moment when we do a search we keep the previous search mode for example images. This behaviour differs from other platform and we want to remove it.

Steps to test this PR:

  1. Search for example “cats"
  2. Move to the image vertical
  3. Search for example “dogs"
  4. Check that when you press enter you are presented the general web results and not the images results.

<!—
Tagging instructions
If this PR isn't ready to be merged for whatever reason it should be marked with the DO NOT MERGE label (particularly if it's a draft)
If it's pending Product Review/PFR, please add the Pending Product Review label.

If at any point it isn't actively being worked on/ready for review/otherwise moving forward (besides the above PR/PFR exception) strongly consider closing it (or not opening it in the first place). If you decide not to close it, make sure it's labelled to make it clear the PRs state and comment with more information.
—>

Internal references:

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

Copy link
Contributor

@tomasstrba tomasstrba left a comment

Choose a reason for hiding this comment

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

LGTM! ✅

@SabrinaTardio SabrinaTardio merged commit 1b4e75d into develop Aug 21, 2023
@SabrinaTardio SabrinaTardio deleted the sabrina/update-addressbar-SERP-logic branch August 21, 2023 11:23
samsymons added a commit that referenced this pull request Aug 22, 2023
# By Diego Rey Mendez (3) and others
# Via Diego Rey Mendez (1) and GitHub (1)
* develop:
  Require Duck Player scheme URL to be passed from YouTube Overlay User Script (#1519)
  extend valid urls to include IPv6 (#1492)
  Fix bug where Back navigation doesn't work properly with bangs (#1510)
  remove logic to keep previous search mode (#1500)
  Bump version to 1.53.0 (52)
  Update embedded files
  Adds a file that's generated when the unit tests are run (#1511)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants