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

Icons not visible with the jupyterlab-night theme #6633

Closed
jtpio opened this issue Nov 21, 2022 · 2 comments · Fixed by #6678
Closed

Icons not visible with the jupyterlab-night theme #6633

jtpio opened this issue Nov 21, 2022 · 2 comments · Fixed by #6678
Milestone

Comments

@jtpio
Copy link
Member

jtpio commented Nov 21, 2022

Opening the issue in this repo since they used to be visible before if I recall correctly.

Trying the latest 7.0.0a9 pre-release on Binder with the jupyterlab-night theme (https://github.com/martinRenou/jupyterlab-night), the icons in the menu are not clearly visible:

image

It might be related to a change in the latest JupyterLab 4.0 pre-releases.

@jtpio
Copy link
Member Author

jtpio commented Nov 21, 2022

This is an upstream issue: jupyterlab/jupyterlab#13461

Leaving open until we update to a new release that fixes.

@jtpio jtpio added this to the 7.0 milestone Nov 21, 2022
@HaudinFlorence
Copy link

HaudinFlorence commented Nov 21, 2022

In PR #13279, new css variables have been introduced in variable.css file both in light and dark themes and used in the relevant files. The goal is to define colors that are independant of other css variables like for instance --jp-warn-colori for jupyter or notebook icons or --jp-brand-colori for the console icon .

/* File or activity icons and switch semantic variables */
  --jp-jupyter-icon-color: var(--md-orange-900);
  --jp-notebook-icon-color: var(--md-orange-700);
  --jp-json-icon-color: var(--md-orange-700);
  --jp-console-icon-background-color: var(--md-blue-700);
  --jp-console-icon-color: white;
  --jp-terminal-icon-background-color: var(--md-grey-800);
  --jp-terminal-icon-color: var(--md-grey-200);
  --jp-text-editor-icon-color: var(--md-grey-700);
  --jp-inspector-icon-color: var(--md-grey-700);
  --jp-switch-color: var(--md-grey-400);
  --jp-switch-true-position-color: var(--md-orange-700);
  --jp-switch-cursor-color: rgba(255, 255, 255, 1);

I guess that for future custom themes these variables should also be present in the variable.css file to properly render the icons.

@RRosio RRosio removed the status:Needs Triage Applied to issues that need triage label Dec 1, 2022
@jtpio jtpio mentioned this issue Jan 5, 2023
10 tasks
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants