Skip to content

fix(browser): SearchView fragment state transitions [dev only]#20399

Open
david-allison wants to merge 1 commit intoankidroid:mainfrom
david-allison:18709-fix-standard-search-overlap
Open

fix(browser): SearchView fragment state transitions [dev only]#20399
david-allison wants to merge 1 commit intoankidroid:mainfrom
david-allison:18709-fix-standard-search-overlap

Conversation

@david-allison
Copy link
Member

@david-allison david-allison commented Mar 5, 2026

Purpose / Description

Previously if opening the 'advanced' search, the content of the 'standard' search overlapped with it

This ensures that a tag is used when attaching the 'standard' search so it can be hidden properly

Fixes

Approach

Always use the tag to create/restore the fragment

How Has This Been Tested?

Personal browser-advanced-more branch

Content no longer overlaps between the two fragments

Checklist

  • You have a descriptive commit message with a short title (first line, max 50 chars).
  • You have commented your code, particularly in hard-to-understand areas
  • You have performed a self-review of your own code
  • UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
  • UI Changes: You have tested your change using the Google Accessibility Scanner

@david-allison david-allison added the Needs Second Approval Has one approval, one more approval to merge label Mar 5, 2026
@david-allison david-allison changed the title fix(browse): SearchView fragment state transitions [dev only] fix(card-browser): SearchView fragment state transitions [dev only] Mar 5, 2026
Previously if opening the 'advanced' search, the content of the
 'standard' search overlapped with it

This ensures that a tag is used when attaching the 'standard' search
 so it can be hidden properly

Issue 18709
@david-allison david-allison force-pushed the 18709-fix-standard-search-overlap branch from 1c6972f to 2fb07f5 Compare March 5, 2026 21:08
@david-allison david-allison changed the title fix(card-browser): SearchView fragment state transitions [dev only] fix(browser): SearchView fragment state transitions [dev only] Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Second Approval Has one approval, one more approval to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant