Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Disable Clear browsing settings when in PB mode #606

Closed
srirambv opened this issue Dec 7, 2018 · 4 comments
Closed

Disable Clear browsing settings when in PB mode #606

srirambv opened this issue Dec 7, 2018 · 4 comments
Labels
blocked: needs info Needs further information before work can commence enhancement QA/Yes release-notes/exclude won't have Requirement which the project team has agreed it will not deliver

Comments

@srirambv
Copy link
Contributor

srirambv commented Dec 7, 2018

Description:

Disable Clear browsing settings when in PB mode otherwise leads to confusion similar to #605

Steps to Reproduce

  1. Open Private tab
  2. Open settings and clear private tab
  3. All settings are shown and doesn't differentiate if its for normal or private tabs

Actual result:
Disable Clear browsing settings when in PB mode

Expected result:
Disable clear private data option when PB mode is switched on

Reproduces how often: [Easily reproduced, Intermittent Issue]
Easy

Brave Version:
All

Device details:
All

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

@kylehickinson
Copy link
Collaborator

By PB mode do you mean "Private Browsing Only" or just when you're currently in a private tab?

@srirambv
Copy link
Contributor Author

During Private browsing only mode

@kylehickinson
Copy link
Collaborator

kylehickinson commented Dec 13, 2018

Turning on private only browsing doesn't erase your history for regular tabs (it even currently keeps your regular tabs—which is a different issue: #580). So the user could definitely still clear their private data when they're browsing private only mode. After they do it once though if they never turn off private only mode then there'd be no more data to remove.

@jhreis jhreis added the blocked: needs info Needs further information before work can commence label Mar 18, 2019
@jhreis jhreis added the won't have Requirement which the project team has agreed it will not deliver label Apr 1, 2019
@jhreis
Copy link
Contributor

jhreis commented Apr 1, 2019

We will be renaming this copy to add more clarity, which should help with this confusion, but this specifically we do not currently want.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
blocked: needs info Needs further information before work can commence enhancement QA/Yes release-notes/exclude won't have Requirement which the project team has agreed it will not deliver
Projects
None yet
Development

No branches or pull requests

5 participants