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

Add Profile Manager items to App menu #5212

Closed
rebron opened this issue Jul 11, 2019 · 8 comments · Fixed by brave/brave-core#3096
Closed

Add Profile Manager items to App menu #5212

rebron opened this issue Jul 11, 2019 · 8 comments · Fixed by brave/brave-core#3096

Comments

@rebron
Copy link
Collaborator

rebron commented Jul 11, 2019

Description

Add Profile Manager items to the hamburger menu.

  1. 'Create New Profile' -> creates a new profile and launches into brave://welcome with that new profile.
  2. 'Open Guest Window' -> launches into a new Guest Window

Design

browser_menu

@rebron rebron added feature/profile-manager priority/P3 The next thing for us to work on. It'll ride the trains. labels Jul 11, 2019
@rebron
Copy link
Collaborator Author

rebron commented Aug 5, 2019

@bradleyrichter Can you confirm final on mock above?

@bradleyrichter
Copy link

@rebron looks good

@petemill
Copy link
Member

petemill commented Aug 6, 2019

@bradleyrichter @rebron can I ask to confirm the ellipses (or lack of) in the mockup.

Create a New Profile… and Open Guest Window (and New X Window for that matter) all do the same thing - open another window, so I'm not sure why Create New Profile would be the only one ending with an ellipsis. Please could you explain?

@rebron
Copy link
Collaborator Author

rebron commented Aug 6, 2019

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).

@petemill
Copy link
Member

petemill commented Aug 6, 2019

@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

@rebron
Copy link
Collaborator Author

rebron commented Aug 6, 2019

@bradleyrichter We should remove the ellipsis in this case then and it should just be Create a New Profile.

@rebron rebron changed the title Add Profile Manager items to (hamburger) menu Add Profile Manager items to App menu Aug 6, 2019
@bsclifton bsclifton added this to the 0.70.x - Nightly milestone Aug 6, 2019
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 23, 2019

@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)

  1. The Create New Profile item available only in Normal and Private/Incognito windows Not in Guest and Tor windows
  2. The Guest Window item available only in Normal and Private/Incognito windows Not in Guest and Tor windows

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 23, 2019

Verification passed on

Brave 0.69.128 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified the test plan from App Menu gets Create Profile and Switch to Guest Profile items brave-core#3096
  • Verified that click on Create a New Profile from the settings menu launches brave://welcome onboarding flow with a new profile
  • Verified that Create New Profile item available only in Normal and Private/Incognito windows Not in Guest and Tor windows
  • Verified that Guest Window item available only in Normal and Private/Incognito windows Not in Guest and Tor windows
  • Verified that Open Guest Window opens the guest window
  • Verified that Create New Profile and Open Guest Window items are in the disabled mode in Guest and Tor windows

Verification passed on

Brave 0.69.128 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS Ubuntu 18.04 LTS
  • Verified the test plan from App Menu gets Create Profile and Switch to Guest Profile items brave-core#3096
  • Verified that click on Create a New Profile from the settings menu launches brave://welcome onboarding flow with a new profile
  • Verified that Create New Profile item available only in Normal and Private/Incognito windows Not in Guest and Tor windows
  • Verified that Guest Window item available only in Normal and Private/Incognito windows Not in Guest and Tor windows
  • Verified that Open Guest Window opens the guest window
  • Verified that Create New Profile and Open Guest Window items are in the disabled mode in Guest and Tor windows

Verified passed with

Brave 0.69.130 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.13.6 (Build 17G5019)
  • Verified the test plan from App Menu gets Create Profile and Switch to Guest Profile items brave-core#3096
  • Verified that click on Create a New Profile from the settings menu launches brave://welcome onboarding flow with a new profile
  • Verified that Create New Profile item available only in Normal and Private/Incognito windows Not in Guest and Tor windows
  • Verified that Guest Window item available only in Normal and Private/Incognito windows Not in Guest and Tor windows
  • Verified that Open Guest Window opens the guest window
  • Verified that Create New Profile and Open Guest Window items are in the disabled mode in Guest and Tor windows

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants