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

Global [role=tooltip] styles impact notebook extensions #5670

Closed
magdapoppins opened this issue Aug 11, 2020 · 3 comments · Fixed by #5672
Closed

Global [role=tooltip] styles impact notebook extensions #5670

magdapoppins opened this issue Aug 11, 2020 · 3 comments · Fixed by #5672

Comments

@magdapoppins
Copy link
Contributor

magdapoppins commented Aug 11, 2020

In fb08a1b#diff-321a1fb8c95f4257fbede791b7856b7cR438-R459 some global styles regarding elements with role="tooltip" were introduced. They were meant to fix issues specific to the .dynamic-buttons-div element, and wouldn't need to be put in global scope.

Setting every element with role="tooltip" visibility to hidden impacts notebook extensions and makes it tedious to use this attribute correctly.

I suggest changing the scope of the above mentioned style to .dynamic-buttons [role=tooltip] in notebook/static/tree/less/tree.less line 438.

@akx
Copy link
Contributor

akx commented Aug 11, 2020

The related issue and PR (so GitHub links them here): #3964, #4729

@Zsailer
Copy link
Member

Zsailer commented Aug 11, 2020

@magdapoppins, good catch! Would you be interested in opening a PR with your proposed fix? Otherwise, I'm happy to submit the change myself. Thanks!

@magdapoppins
Copy link
Contributor Author

@Zsailer I can open a pr tomorrow. 👍

magdapoppins added a commit to magdapoppins/notebook that referenced this issue Aug 12, 2020
- Change the scope of styles specific to `.dynamic-buttons` to that class only (introed in jupyter#4729)
- Drop the width style from tooltips since `display: inline` makes it have no effect

Fixes jupyter#5670
magdapoppins added a commit to magdapoppins/notebook that referenced this issue Aug 12, 2020
- Change the scope of styles specific to `.dynamic-buttons` to that class only (introed in jupyter#4729)
- Drop the width style from tooltips since `display: inline` makes it have no effect

Fixes jupyter#5670
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants