-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Different private profile button style #8807
Comments
cc: @tomlowenthal |
cc: @karenkliu can you take a look? This is on the latest nightly https://github.com/brave/brave-browser/releases/tag/v1.8.32 |
No opinion on the order of the elements. But this is the first time I learned that this was a button rather than a badge. As far as I know, this was never specified as a button, and I’d prefer if it went back to being a badge. |
☝️ what @tomlowenthal said. This order [icon-text] is correct: But it should have the same background color as the toolbar and should not be clickable. |
This is a nice feature; what action should we take with this ticket? Is the problem that it looks like a button? |
@mkarolin I see - thanks for pointing this out. @bsclifton I suspect that the "Private" label became clickable after Chrome added some additional functionality and we inherited it. For private windows the issue is styling. This is what I recommend: Style the "Private" button like a Brave button so users know there is functionality there On click, show a panel tweaked so the Private icon doesn't look like another button and with Brave styling + icons: When multiple windows are open, show this text treatment: We should support dark theme for the panel: TOR windows also inherits this functionality from Chrome but it doesn't really work:
Should we file a separate issue to fix this in Tor? I would recommend doing the same treatment as our Private windows to keep interactions consistent; also keeping the sunglasses icon for Tor windows: Figma link for reference: https://www.figma.com/file/5THkuEtO2Ewn9LfqrHZP9a/Desktop-Private-Tabs-and-TOR?node-id=4%3A4737 |
From current master build(
Version 1.8.31 Chromium: 81.0.4044.69 (Developer Build) (64-bit)
):From current beta build(
Version 1.7.67 Chromium: 80.0.3987.149 (Official Build) beta (64-bit)
):The text was updated successfully, but these errors were encountered: