-
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
Add Profile Manager items to App menu #5212
Comments
@bradleyrichter Can you confirm final on mock above? |
@rebron looks good |
@bradleyrichter @rebron can I ask to confirm the ellipses (or lack of) in the mockup.
|
I believe it means more options will be showing and more info will be needed whereas without the ellipses a user will be going to that thing and no other info is needed. So with create a new profile... the more info needed is select an icon and edit a name (or just use the default name provided). |
@rebron sure, but in this case the user is not prompted for more information, the new profile simply opens. See the existing macOS system menu Profile - Add Profile option |
@bradleyrichter We should remove the ellipsis in this case then and it should just be Create a New Profile. |
@petemill brave/brave-core#3096 - Would need some clarifications regarding the test plan, should it be as mentioned below? Please let me know if I am missing anything in testing notes #5212 (comment)
|
Verification passed on
Verification passed on
Verified passed with
|
Description
Add Profile Manager items to the hamburger menu.
Design
The text was updated successfully, but these errors were encountered: