-
Notifications
You must be signed in to change notification settings - Fork 949
Upgrade to Font-Awesome v5.x #2217
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
Comments
Thanks for filing this, and for noting the incompatibility! |
Maybe now would be a good time to update fontawesome @jasongrout ? |
Do you know if there were any backwards-incompatible name changes? I think it's great to look at upgrading fontawesome. |
According to what @DougRzz is saying, yes, there seem to be name changes :) But I guess it's fine as the next ipywidgets version will be backward-incompatible. |
Ah, right, sorry, I should have read the whole thread :). That's unfortunate, but sure, I guess now is the time to make the change. |
Perhaps this can be addressed while at it? #2477 |
@martinRenou and I discussed this. The current status quo:
So, the question becomes: do we namespace the entire FA5 library and ship it with controls to have a known version? Or do we just leave this unspecified and up to the environment? |
Also, FA5 includes a shim css file that maps v4 names to v5, so apparently it doesn't have to be backwards incompatible. |
One upside of leaving this responsibility to the environment is the possibility to use a pro version of FA. If we ship the free version of FA, the user is stuck with it and can't use their license. |
So Font-Awesome is upgraded in JupyterLab and the HTMLManager, see jupyterlab/jupyterlab#7769 and #2713. Now we might want to discuss upgrading in Jupyter Notebook :) |
Closing as resolved. Upgrading the notebook can be a ticket for that repo. Thanks again! |
Upgrade Font Awesome from v4.7 to v5.3. There are twice as many free icons in v5.3 so it would be a useful upgrade.
However, this would be a breaking change because some of the names have changed. e.g pie-chart (https://fontawesome.com/v4.7.0/icon/pie-chart ) vs. chart-pie (https://fontawesome.com/icons/chart-pie?style=solid).
The text was updated successfully, but these errors were encountered: