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

Jupyter 5.1 changes add_buttons_group #1102

Closed
Sukneet opened this issue Sep 19, 2017 · 10 comments
Closed

Jupyter 5.1 changes add_buttons_group #1102

Sukneet opened this issue Sep 19, 2017 · 10 comments

Comments

@Sukneet
Copy link
Contributor

Sukneet commented Sep 19, 2017

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:

image

@jcb91
Copy link
Member

jcb91 commented Sep 19, 2017

D'oh!

@Sukneet
Copy link
Contributor Author

Sukneet commented Sep 19, 2017

The following seems to work
Sukneet@1547b3e

Is the style ok?

@jcb91
Copy link
Member

jcb91 commented Sep 19, 2017

Looks good to me, and the actions have the added bonus that they can be used in the command palette :)

@jcb91
Copy link
Member

jcb91 commented Sep 20, 2017

Opened a PR #1103. There are quite a few other nbextensions that will need similar updates, though. Thanks @Sukneet for reporting & fixing freeze 😄 🙇

@Sukneet
Copy link
Contributor Author

Sukneet commented Sep 20, 2017

I fixed the other ones last night. Just need to verify they are all working.

@juhasch
Copy link
Member

juhasch commented Sep 20, 2017

Thanks for working on this. I am updating n the runtools extension and fixed it there already.

@Sukneet
Copy link
Contributor Author

Sukneet commented Sep 21, 2017

I think I got them all (except runtools)
Sukneet@b7aa1bd

@takluyver
Copy link

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:

https://github.com/takluyver/cite2c/blob/f310fb00af829664c756404c343d948ab556a2e0/cite2c/nbext/main.js#L296-L302

Produces buttons like this:

screenshot from 2017-10-05 11-30-15

@jcb91
Copy link
Member

jcb91 commented Oct 16, 2017

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 🤣

@jcb91 jcb91 closed this as completed Oct 16, 2017
@kenptr
Copy link

kenptr commented Mar 14, 2018

screen shot 2018-03-14 at 12 20 14 am

2to3 and code_prettify still have long labels.

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

No branches or pull requests

5 participants