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

Update for Jupyterlab 2 #30

Merged
merged 7 commits into from
Mar 18, 2020
Merged

Update for Jupyterlab 2 #30

merged 7 commits into from
Mar 18, 2020

Conversation

manics
Copy link
Owner

@manics manics commented Mar 8, 2020

Updates the lab extension for Jupyterlab 2.

Test this branch on mybinder: https://mybinder.org/v2/gh/manics/jupyter-offlinenotebook/jupyterlab2?urlpath=lab%2Ftree%2Fexample.ipynb

- Switch to Lumino
- PartialJSONValue instead of JSONValue
- iconName instead of iconClass
- panel.content.model may be null
@manics
Copy link
Owner Author

manics commented Mar 8, 2020

image
Font-awesome icons are missing

@manics
Copy link
Owner Author

manics commented Mar 9, 2020

Using the Firefox dev-tools I've found there's a CSS class .jp-Button.minimal that sets color: unset !important; and effectively makes the icon invisible. Disabling this class in dev-tools makes the icon appear:

image

The class is defined in Jupyterlab here:
https://github.com/jupyterlab/jupyterlab/blob/91d4abf003836e01c1f3e5b2541aba3f6740e94e/packages/ui-components/style/base.css#L70-L72
and I think this is where it's added to the toolbar button:
https://github.com/jupyterlab/jupyterlab/blob/91d4abf003836e01c1f3e5b2541aba3f6740e94e/packages/apputils/src/toolbar.tsx#L496

@manics manics force-pushed the jupyterlab2 branch 2 times, most recently from 52b7b28 to d3e590f Compare March 9, 2020 23:52
@manics
Copy link
Owner Author

manics commented Mar 14, 2020

The Icons are visible now- tested in Firefox and Chrome.

package.json Show resolved Hide resolved
@manics manics merged commit 9a31569 into master Mar 18, 2020
@manics manics deleted the jupyterlab2 branch March 18, 2020 20:08
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 this pull request may close these issues.

3 participants