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
move contextual help to the help menu #6678
Conversation
Thanks for making a pull request to JupyterLab! To try out this branch on binder, follow this link: |
+1 to moving it to the help menu. Can we make it a separate group just after the About and Launch classic? I think it really is different than those items. So the menu would be:
(Notice I also rearranged the jlab/jupyter reference group - I think this order makes more sense, to put the jlab stuff first) |
Hmmm---and thinking about it, most of those items are "open" actions. It's a bit weird that only one of them says "Open". @ellisonbg, thoughts? |
Discussed this in person with @ellisonbg, who was 50-50 on remove or keeping "Open " in the menu name. At first I was going to keep it, but then I figured that in the command palette, type "open" would select the contextual help as the first item, which is not a good user experience, so I dropped open, here's what it looks like now (and as a bonus, contextual help is fairly high in the command listing with an empty entry, so it's somewhat discoverable that way, too. We will definitely want to let users know about this rename and move of the menu item in the changelog. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ivanov, like it needs a rebase.
@@ -157,6 +160,13 @@ function activate( | |||
command => ({ command }) | |||
); | |||
helpMenu.addGroup(labGroup, 0); | |||
|
|||
// Contextual help in its own group | |||
const contextualHelpGroup = [inspector ? 'inspector:open' : null].map( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this check. We should do more of these.
067f597
to
5936a19
Compare
Looks like a legitimate test failure. |
@saulshanabrook was concerned about the |
This works even if the inspector is not available. If you want to change the |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related discussion. |
No description provided.