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

Fix icons in example applications #7800

Merged
merged 10 commits into from Jan 20, 2020
Merged

Fix icons in example applications #7800

merged 10 commits into from Jan 20, 2020

Conversation

afshin
Copy link
Member

@afshin afshin commented Jan 20, 2020

@tchaikovskyInd_twitter reported on the Gitter channel that the example notebook application was not creating toolbar buttons and the console log had this error: SVG HTML was malformed for JLIcon instance. This PR fixes the webpack configuration of the different apps to fix this issue.

Code changes

Only a configuration change for webpack.

User-facing changes

Broken toolbars in example apps work now.

Backwards-incompatible changes

NA

@afshin afshin added the bug label Jan 20, 2020
@afshin afshin added this to the 2.0 milestone Jan 20, 2020
@afshin afshin self-assigned this Jan 20, 2020
@jupyterlab-dev-mode
Copy link

Thanks for making a pull request to JupyterLab!

To try out this branch on binder, follow this link: Binder

@afshin
Copy link
Member Author

afshin commented Jan 20, 2020

cc: @telamonian

@afshin
Copy link
Member Author

afshin commented Jan 20, 2020

This PR makes the examples compatible with #7700

Copy link
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@blink1073 blink1073 merged commit f17f59c into jupyterlab:master Jan 20, 2020
@lock lock bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Feb 21, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Feb 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. tag:Examples tag:Testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants