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

[Bug] Deleting opened tabs, exits Settings-Delete browsing data and returns to homescreen #4807

Closed
lobontiumira opened this issue Aug 19, 2019 · 7 comments
Assignees
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified eng:ready Ready for engineering Feature:Settings P3 Some future sprint S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist
Projects

Comments

@lobontiumira
Copy link

lobontiumira commented Aug 19, 2019

Steps to reproduce

  1. Make sure to have at least 1 tab opened.
  2. Go to Settings - Delete browsing data - select Open tabs and delete.

Expected behavior

The user should remain in Settings - Delete browsing data page, exactly like it does when deleting "Browsing history" and "Collections". Maybe it is a consequence of #4376.

Actual behavior

The user is redirected to the homescreen.

Device information

  • Android device: Huawei P9 Lite (Android 6), Xiaomi Mi 8 Lite (Android 9)
  • Fenix version: RC 1.3.0-rc.3

┆Issue is synchronized with this Jira Task

@lobontiumira lobontiumira added 🐞 bug Crashes, Something isn't working, .. S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist Feature:Settings labels Aug 19, 2019
@mikeGolf08
Copy link

#4788 & #4642

@colintheshots
Copy link
Contributor

colintheshots commented Aug 19, 2019

I had UX sign off for this behavior in the case of deleted tabs. This is expected behavior. The problem is that the user could've left a deleted tab in the back stack that is invalid and the navigation library doesn't allow removing fragments from the back stack.

@lobontiumira
Copy link
Author

@colintheshots just to be clear: when deleting "Browsing history" and "Collections" the user remains in the Settings-Delete browsing data page, but when deleting Open tabs, the user is redirected to the home screen?
Thank you!

@boek boek added b:navigation P3 Some future sprint labels Dec 28, 2019
@boek boek added this to App Navigation in Workflow Feb 5, 2020
@topotropic
Copy link

@AmyYLee I think you worked on it, could you have a look and see if it's expected behavior? Thanks!

@AmyYLee
Copy link
Collaborator

AmyYLee commented Feb 12, 2020

@softvision-miralobontiu @colintheshots

To be clear, deleting any listed items will take the user back to the Settings main screen.

@ekager ekager added the eng:ready Ready for engineering label Feb 12, 2020
@mcarare
Copy link
Contributor

mcarare commented Feb 17, 2020

It currently navigates to home only if you open settings from the browser screen, as the current tab is closed.

@mcarare mcarare added eng:qa:needed QA Needed and removed eng:qa:needed QA Needed labels Feb 17, 2020
@mcarare mcarare moved this from App Navigation to Doing in Workflow Feb 17, 2020
mcarare pushed a commit to mcarare/fenix that referenced this issue Feb 17, 2020
@mcarare mcarare self-assigned this Feb 18, 2020
mcarare pushed a commit to mcarare/fenix that referenced this issue Feb 18, 2020
mcarare pushed a commit to mcarare/fenix that referenced this issue Feb 18, 2020
mcarare pushed a commit to mcarare/fenix that referenced this issue Feb 18, 2020
mcarare pushed a commit to mcarare/fenix that referenced this issue Feb 18, 2020
mcarare pushed a commit to mcarare/fenix that referenced this issue Feb 18, 2020
boek added a commit to boek/fenix that referenced this issue Feb 18, 2020
@boek boek added the eng:qa:needed QA Needed label Feb 18, 2020
@abodea
Copy link
Member

abodea commented Feb 19, 2020

Verified as fixed on the latest Beta build 4.0.0-beta.1 with Google Pixel 3a XL (Android 10).
Note that now the redirect is correctly made to the Settings main screen after deleting any listed items.

@abodea abodea closed this as completed Feb 19, 2020
@abodea abodea added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Feb 19, 2020
@sblatz sblatz moved this from Doing to Done in Workflow Feb 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified eng:ready Ready for engineering Feature:Settings P3 Some future sprint S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist
Projects
No open projects
Workflow
  
Done
Development

No branches or pull requests

9 participants