-
Notifications
You must be signed in to change notification settings - Fork 812
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
Jupyter 5.1 changes add_buttons_group #1102
Comments
D'oh! |
The following seems to work Is the style ok? |
Looks good to me, and the actions have the added bonus that they can be used in the command palette :) |
I fixed the other ones last night. Just need to verify they are all working. |
Thanks for working on this. I am updating n the |
I think I got them all (except runtools) |
Sorry for this - I thought I had been more careful in making the change than I actually was. Thanks for working on adapting jupyter_contrib_nbextensions to deal with this change. Once PR #1103 has cleared up the problematically long labels, I hope that some nbextensions will be able to make judicious use of shorter labels on the buttons to make their purpose clearer. E.g. this code in cite2c: Produces buttons like this: |
Ok,this should be fixed by #1103 & #1123. As @takluyver notes, though, it might be a good idea to add labels to some of our more ambiguous buttons. We could even make the button labels configurable 🤣 |
Juypter commit jupyter/notebook@af244a6 adds an html span with the label value when calling add_buttons_group. This causes a long name in the toolbar for a lot of extensions. For example, the freeze extension:
The text was updated successfully, but these errors were encountered: