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] Settings page for guest and Tor windows are empty #5065

Closed
simonhong opened this issue Jun 28, 2019 · 5 comments · Fixed by brave/brave-core#5985
Closed

[Desktop] Settings page for guest and Tor windows are empty #5065

simonhong opened this issue Jun 28, 2019 · 5 comments · Fixed by brave/brave-core#5985
Assignees
Labels
feature/global-settings Settings at browser level independent of shields settings feature/settings needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/exclude

Comments

@simonhong
Copy link
Member

simonhong commented Jun 28, 2019

Can repro all channels from master(0.69.x) to stable (0.65.x)
Settings page of guest window should have search engine part.

Note: master debug build shows search engine part properly.

Test Plan

  • Open a guest window
  • Visit chrome://settings
  • Page should show
  • The sections that should be visible are: search, shields and social blocking
@tildelowengrimm tildelowengrimm added feature/global-settings Settings at browser level independent of shields settings feature/settings feature/tor feature/tor/guest-semantics Exposing the fact that private windows with Tor are built on guest sessions. needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. and removed feature/tor feature/tor/guest-semantics Exposing the fact that private windows with Tor are built on guest sessions. labels Jul 2, 2019
@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Jul 5, 2019
@btlechowski
Copy link

Still reproducible

Brave 1.1.22 Chromium: 79.0.3945.79 (Official Build) (64-bit)
Revision 29f75ce3f42b007bd80361b0dfcfee3a13ff90b8-refs/branch-heads/3945@{#916}
OS Ubuntu 18.04 LTS

image

cc @brave/legacy_qa

@btlechowski
Copy link

Marking as QA/Blocked - need a new build to test, 1.11.92 or newer.

@LaurenWags
Copy link
Member

Removing QA/Blocked, https://github.com/brave/brave-browser/releases/tag/v1.11.92 is available

@LaurenWags
Copy link
Member

LaurenWags commented Jul 8, 2020

@petemill is there a test plan for this one? brave://settings page now displays on a guest window, but I'm not sure which sections should be available. Currently Shields, Social media blocking, and Search engine settings are accessible - is this ok? Should there be more settings or fewer settings?

Also, brave://settings does not display for a Tor window - should it? (per issue title, Tor is mentioned as well)

Any extra info for QA on what we're trying to fix/accomplish with this issue would be helpful, thanks! cc @rebron @kjozwiak @bsclifton

re-adding QA/Blocked until this is sorted.

@kjozwiak
Copy link
Member

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)

Screen Shot 2020-07-08 at 5 40 49 PM


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 clicking on Settings via the hamburger menu in a Guest window correctly opened brave://settings
    • ensured that typing brave://settings into the URL in a Guest window correctly opened the settings page
  • ensured that only Shields, Social media blocking and Search engine were visible.
  • ensured that clicking on the three options on the navigation list correctly focuses the correct section

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 clicking on Settings via the hamburger menu in a Guest window correctly opened brave://settings
    • ensured that typing brave://settings into the URL in a Guest window correctly opened the settings page
  • ensured that only Shields, Social media blocking and Search engine were visible.
  • ensured that clicking on the three options on the navigation list correctly focuses the correct section

image

@rebron rebron changed the title Settings page of guest(also tor) is empty [Desktop] Settings page for guest and Tor windows are empty Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/global-settings Settings at browser level independent of shields settings feature/settings needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/exclude
Projects
None yet
8 participants