-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Disable opening Tor in guest window #7237
Comments
@yrliou looks like the option for
|
@LaurenWags this behavior should be good 😄 I believe this is properly fixed in 1.4 with brave/brave-core#4235 (see screenshots) |
@LaurenWags There are something that my PR was built upon which is only in 1.3 and later. I only uplift my own PR to make changes minimum, we should be fine in later versions. |
thanks for confirming @bsclifton and @yrliou 😄 |
Verification passed on
Verified test plan from brave/brave-core#4155 Verified passed with
Verification passed on
|
Description
Currently UI items for opening private window in a guest window is disabled, we should behave the same to not allow opening Tor window in a guest window too to be consistent.
Steps to Reproduce
Actual result:
Hamburger menu:
File menu (macOS):
Context menu:
Expected result:
Hamburger menu:
File menu (macOS):
Context menu:
Reproduces how often:
Always.
Brave version (brave://version info)
For all versions after 1.1.x.
The text was updated successfully, but these errors were encountered: