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

[Desktop] Remove Sync from brave://settings under Tor/Guest Windows #2995

Closed
kjozwiak opened this issue Jan 16, 2019 · 2 comments · Fixed by brave/brave-core#5985
Closed

Comments

@kjozwiak
Copy link
Member

Description

We should remove the Sync option from brave://settings for Tor/Guest windows as sync shouldn't be used while Tor/Guest mode is being used. Preferences that are accessed via the hamburger menu or the file menu are opened in regular tabs but this prevents users from accessing the preferences via brave://settings and clicking on brave://sync.

Steps to Reproduce

  1. launch brave (0.60.13 Chromium: 72.0.3626.53 in this instance)
  2. open a Tor/Guest window and visit brave://settings
  3. notice that the Sync area is being displayed/accessible

Actual result:

screen shot 2019-01-16 at 6 48 24 pm

Expected result:

Sync section under brave://settings shouldn't be visible/accessible under Tor/Guest

Reproduces how often:

100% Reproducible

Brave version (brave://version info)

Brave 0.60.13 Chromium: 72.0.3626.53 (Official Build) dev(64-bit)
Revision 98434e6cd182d68ce396daa92e9c6310422e6763-refs/branch-heads/3626@{#620}
OS Mac OS X

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?

Reproducible wherever sync is enabled.

Additional Information

@kjozwiak
Copy link
Member Author

kjozwiak commented Jul 9, 2020

Just a heads up to @brave/legacy_qa, the Tor case isn't valid anymore as we don't open brave://settings in Tor windows. When a user attempts to open brave://settings in Tor, we either change focus on a tab that already has the settings page opened or create a new brave://settings tab in a normal window.

@kjozwiak
Copy link
Member Author

kjozwiak commented Jul 9, 2020

Verification PASSED on macOS 10.15.5 x64 using the following build:

Brave | 1.11.92 Chromium: 84.0.4147.68 (Official Build) dev (64-bit)
-- | --
Revision | e7b7234037639b407e69c5428c891ce018cd6c0d-refs/branch-heads/4147@{#772}
OS | macOS Version 10.15.5 (Build 19F101)
  • ensured that Sync isn't available under the Hamburger menu under Guest windows
  • ensured that Sync isn't visible under brave://settings under Guest windows

Screen Shot 2020-07-09 at 12 47 25 AM

Screen Shot 2020-07-09 at 12 47 37 AM


Verification passed on


Brave | 1.11.92 Chromium: 84.0.4147.68 (Official Build) dev (64-bit)
-- | --
Revision | e7b7234037639b407e69c5428c891ce018cd6c0d-refs/branch-heads/4147@{#772}
OS | Windows 10 OS Version 1903 (Build 18362.900)


  • ensured that Sync isn't available under the Hamburger menu under Guest windows
  • ensured that Sync isn't visible under brave://settings under Guest windows
    image
    image

Verification passed on

Brave 1.11.92 Chromium: 84.0.4147.68 (Official Build) dev (64-bit)
Revision e7b7234037639b407e69c5428c891ce018cd6c0d-refs/branch-heads/4147@{#772}
OS Ubuntu 18.04 LTS
  • ensured that Sync isn't available under the Hamburger menu under Guest windows
  • ensured that Sync isn't visible under brave://settings under Guest windows
    image
    image

@rebron rebron changed the title Removing Sync from brave://settings under Tor/Guest Windows [Desktop] Remove Sync from brave://settings under Tor/Guest Windows Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment