Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove topbar and non-search settings from brave://settings for guest windows #15053

Closed
emerick opened this issue Mar 31, 2021 · 4 comments
Closed
Labels
closed/wontfix feature/global-settings Settings at browser level independent of shields settings OS/Desktop priority/P4 Planned work. We expect to get to it "soon".

Comments

@emerick
Copy link
Contributor

emerick commented Mar 31, 2021

Description

When opening brave://settings in a guest window, we shouldn't show the topbar or non-search related settings.

Steps to Reproduce

  1. Launch Brave
  2. Hamburger menu -> Open Guest Window
  3. From the newly opened Guest window: Hamburger menu -> Settings

Actual result:

We show a navigation topbar in Guest windows that includes features that are inaccessible from a Guest window (e.g., Brave Rewards) and we show settings that aren't

Expected result:

We shouldn't show the navigation topbar in Guest windows (especially since it has things like "Brave Rewards" which require a regular profile and will crash in non-regular profiles). In addition, we should only show search-related settings here (as Chrome does).

@rebron suggested it should look like this instead:

screen_shot_2021-03-31_at_12 43 44_pm

Reproduces how often:

Easily reproduced.

Desktop Brave version:

Latest master.

Android Device details:

  • Install type (ARM, x86):
  • Device type (Phone, Tablet, Phablet):
  • Android version:

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the nightly channel?

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

@rebron rebron added feature/global-settings Settings at browser level independent of shields settings priority/P4 Planned work. We expect to get to it "soon". labels Apr 5, 2021
@rebron
Copy link
Collaborator

rebron commented Apr 13, 2021

cc: @spylogsster Can you take a look at this one as well since you were in here already via: brave/brave-core#8445

@jcsavage2
Copy link

Hello! I would like to investigate this issue. is it still available and are you looking for help? I'd love to give this a shot!

@rebron
Copy link
Collaborator

rebron commented Jul 28, 2023

We can probably change this to wontfix. wdyt @emerick? The top navigation in guest doesn't have Rewards there and bookmarks and history show a proper warning.

Other settings for Shields and Social Media blocking seem fine.

@emerick
Copy link
Contributor Author

emerick commented Jul 31, 2023

Yes, wontfix makes sense for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/wontfix feature/global-settings Settings at browser level independent of shields settings OS/Desktop priority/P4 Planned work. We expect to get to it "soon".
Projects
None yet
Development

No branches or pull requests

4 participants