Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Dismiss search dialog when interacting with popups from homescreen #26790

Closed
Alexandru2909 opened this issue Sep 2, 2022 · 1 comment · Fixed by #26832, fork-house/fenix#13, nathanmkaya/fenix#108, C-EO/fenix#4 or fork-house/fenix#14
Assignees
Labels
eng:qa:verified QA Verified eng:tech-debt Engineering debt. Missing unit tests, etc.
Milestone

Comments

@Alexandru2909
Copy link
Contributor

Alexandru2909 commented Sep 2, 2022

In #26690 we noticed that the user could not interact with the remove dropdown for recent synced tabs from the homescreen when SearchDialog was displayed. This is still an issue with other homescreen sections which support popups or dropdown menus: Recent tabs, Recently visited, Recently saved.

┆Issue is synchronized with this Jira Task

@Alexandru2909 Alexandru2909 added the eng:tech-debt Engineering debt. Missing unit tests, etc. label Sep 2, 2022
@github-actions github-actions bot added the needs:triage Issue needs triage label Sep 2, 2022
@Alexandru2909 Alexandru2909 removed the needs:triage Issue needs triage label Sep 2, 2022
@Alexandru2909 Alexandru2909 self-assigned this Sep 2, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Sep 6, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Sep 6, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Sep 6, 2022
@github-actions github-actions bot added the eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged label Sep 6, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Sep 7, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Sep 7, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Sep 7, 2022
Mugurell pushed a commit to Alexandru2909/fenix that referenced this issue Sep 14, 2022
Mugurell pushed a commit to Alexandru2909/fenix that referenced this issue Sep 14, 2022
Mugurell pushed a commit to Alexandru2909/fenix that referenced this issue Sep 14, 2022
@mergify mergify bot closed this as completed in #26832 Sep 14, 2022
mergify bot pushed a commit that referenced this issue Sep 14, 2022
mergify bot pushed a commit that referenced this issue Sep 14, 2022
@github-actions github-actions bot reopened this Sep 14, 2022
@github-actions github-actions bot added eng:qa:needed QA Needed and removed eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged labels Sep 14, 2022
@github-actions github-actions bot added this to the 106 milestone Sep 14, 2022
@SoftVision-LorandJanos
Copy link

Verified as fixed on the latest Nightly 106.0a1 (2022-09-14T17...) build.
Device used: google Pixel 4 (Android 12).
Closing the ticket as verified.

@SoftVision-LorandJanos SoftVision-LorandJanos added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Sep 15, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 3, 2022
… recent bookmark dropdown menu"

This reverts commit 262aa16.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 12, 2022
… recent bookmark dropdown menu"

This reverts commit 262aa16.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 12, 2022
… recent visit dropdown menu"

This reverts commit b93b085.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 12, 2022
… recent tab dropdown menu"

This reverts commit 44b71bb.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 24, 2022
… recent bookmark dropdown menu"

This reverts commit 262aa16.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 24, 2022
… recent visit dropdown menu"

This reverts commit b93b085.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 24, 2022
… recent tab dropdown menu"

This reverts commit 44b71bb.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 26, 2022
… recent bookmark dropdown menu"

This reverts commit 262aa16.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 26, 2022
… recent visit dropdown menu"

This reverts commit b93b085.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Oct 26, 2022
… recent tab dropdown menu"

This reverts commit 44b71bb.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Nov 15, 2022
… recent bookmark dropdown menu"

This reverts commit 262aa16.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Nov 15, 2022
… recent visit dropdown menu"

This reverts commit b93b085
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Nov 15, 2022
… recent tab dropdown menu"

This reverts commit 44b71bb.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Nov 16, 2022
… recent bookmark dropdown menu"

This reverts commit 262aa16.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Nov 16, 2022
… recent visit dropdown menu"

This reverts commit b93b085
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Nov 16, 2022
… recent tab dropdown menu"

This reverts commit 44b71bb.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Dec 6, 2022
… recent bookmark dropdown menu"

This reverts commit 262aa16.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Dec 6, 2022
… recent visit dropdown menu"

This reverts commit b93b085
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Dec 6, 2022
… recent tab dropdown menu"

This reverts commit 44b71bb.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Dec 13, 2022
… recent bookmark dropdown menu"

This reverts commit 262aa16.
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Dec 13, 2022
… recent visit dropdown menu"

This reverts commit b93b085
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Dec 13, 2022
… recent tab dropdown menu"

This reverts commit 44b71bb.
mergify bot added a commit that referenced this issue Dec 13, 2022
…27262)

* Revert "For #26790 - Dismiss search dialog when opening recent bookmark dropdown menu"

This reverts commit 262aa16.

* Revert "For #26790 - Dismiss search dialog when opening recent visit dropdown menu"

This reverts commit b93b085

* Revert "For #26790 - Dismiss search dialog when opening recent tab dropdown menu"

This reverts commit 44b71bb.

* Revert "For #26690 - Dismiss search dialog when opening recent synced tab dropdown menu"

This reverts commit bda817a.

* For #26957 - Remove code to dismiss search dialog when interacting with homescreen top sites

* For #26957 - Remove code to dismiss search dialog when interacting with homescreen collection

* For #26957 - Remove code to dismiss search dialog when interacting with homescreen recent visits

* For #26957 - Remove code to dismiss search dialog when interacting with homescreen recent tabs

* For #26957 - Remove code to dismiss search dialog when interacting with homescreen recent bookmarks

* For #26957 - Remove code to dismiss search dialog when interacting with pocket stories

* For #26957 - Dismiss search dialog when interacting with home fragment

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
…racting with home fragment (mozilla-mobile/fenix#27262)

* Revert "For mozilla-mobile/fenix#26790 - Dismiss search dialog when opening recent bookmark dropdown menu"

This reverts commit 5caee27.

* Revert "For mozilla-mobile/fenix#26790 - Dismiss search dialog when opening recent visit dropdown menu"

This reverts commit b2af04c

* Revert "For mozilla-mobile/fenix#26790 - Dismiss search dialog when opening recent tab dropdown menu"

This reverts commit bfdf6d2.

* Revert "For mozilla-mobile/fenix#26690 - Dismiss search dialog when opening recent synced tab dropdown menu"

This reverts commit 86d44a4.

* For mozilla-mobile/fenix#26957 - Remove code to dismiss search dialog when interacting with homescreen top sites

* For mozilla-mobile/fenix#26957 - Remove code to dismiss search dialog when interacting with homescreen collection

* For mozilla-mobile/fenix#26957 - Remove code to dismiss search dialog when interacting with homescreen recent visits

* For mozilla-mobile/fenix#26957 - Remove code to dismiss search dialog when interacting with homescreen recent tabs

* For mozilla-mobile/fenix#26957 - Remove code to dismiss search dialog when interacting with homescreen recent bookmarks

* For mozilla-mobile/fenix#26957 - Remove code to dismiss search dialog when interacting with pocket stories

* For mozilla-mobile/fenix#26957 - Dismiss search dialog when interacting with home fragment

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.