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

Spills styles over even when not active #32

Closed
krassowski opened this issue Jun 7, 2021 · 6 comments · Fixed by #33
Closed

Spills styles over even when not active #32

krassowski opened this issue Jun 7, 2021 · 6 comments · Fixed by #33

Comments

@krassowski
Copy link

Hi @telamonian, just FYI, there is a strange issue with this theme (or all external themes? I do not know) which results in some styles being active even if the theme is not active: https://stackoverflow.com/questions/67870994/change-the-mouse-select-color-in-jupyterlab-3

Even more strange is that simply disabling it via jupyter labextension disable @telamonian/theme-darcula did not work for me, I had to uninstall it completely with pip uninstall theme-darcula to get rid of the issue (but maybe this is a federated extensions thing).

@telamonian
Copy link
Owner

well that sounds like a horrible bug.

A) what browser?

B) have you tested this out with any other theme?

@krassowski
Copy link
Author

A) The user who reported it on Stack Overflow used, quote:

Jupyter lab is version v3.0.16. The complete stack is Jupyterhub with dockerspawner on Ubuntu 20.04.2. I am using Google Chrome browser on Win 10, the same problem happens in Edge

I verified it locally on Google Chrome.

B) I did not - don't use themes other than the built-in ones.

@krassowski
Copy link
Author

It seems to be coming from https://github.com/telamonian/theme-darcula/blob/master/style/codemirror-darcula.css and the comment in there:

/*
* Codemirror styling that isn't currently supported by Jupyterlab themes
*/

Makes me suspect it might be the culprit.

@telamonian
Copy link
Owner

yeah that's the source, but the entire stylesheet should get unloaded on theme switch. Something's getting in the way

@jtpio
Copy link
Contributor

jtpio commented Jun 9, 2021

@telamonian @krassowski should we move this conversation to the lab issue? jupyterlab/jupyterlab#10378

It seems to also be affecting the miami nights theme (linked in the issue), so probably the issue is related to federated theme extensions in general?

@jasongrout
Copy link
Contributor

Fix over in #33

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

Successfully merging a pull request may close this issue.

4 participants