-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Change Password Manager tab icon to match Brave brand #33242
Comments
cc: @aguscruiz |
@fallaciousreasoning we should use the same one we have in the menu here 👀 |
The one in the menu looks a little odd next to our other icons, as it doesn't have much height and looks a wee bit small: Do you think the |
Ugh, I didn't know you could install it as a web app and it'd look THAT big hah. I think the one in the tab is ok. But I'll have to spend more time with the big one. I'll put it in my backlog for now, is that ok? |
@aguscruiz's much nicer icon: |
Description
The icon on the password manager tab looks like it was pulled straight from Chrome/Chromium — even through Chrome's does look slightly different (both shown below). The icon in the tab should match the rest of the Brave brand.
Icon in Brave:

Icon in Chrome:

The text was updated successfully, but these errors were encountered: