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

Upgrade to Font-Awesome v5.x #2217

Closed
DougRzz opened this issue Sep 19, 2018 · 11 comments
Closed

Upgrade to Font-Awesome v5.x #2217

DougRzz opened this issue Sep 19, 2018 · 11 comments
Assignees
Labels
resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Milestone

Comments

@DougRzz
Copy link
Contributor

DougRzz commented Sep 19, 2018

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).

@jasongrout jasongrout added this to the Major release milestone Sep 19, 2018
@jasongrout
Copy link
Member

Thanks for filing this, and for noting the incompatibility!

@martinRenou
Copy link
Member

Maybe now would be a good time to update fontawesome @jasongrout ?

@jasongrout
Copy link
Member

Do you know if there were any backwards-incompatible name changes?

I think it's great to look at upgrading fontawesome.

@martinRenou
Copy link
Member

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.

@jasongrout
Copy link
Member

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.

@rskabelund
Copy link

Perhaps this can be addressed while at it? #2477

@jasongrout
Copy link
Member

@martinRenou and I discussed this. The current status quo:

  1. the widget manager is responsible for making sure font-awesome on the page
  2. the widget manager may already put font-awesome 5 on the page (or be in an environment where fa5 is on the page). Luckily, all the environments that we know about have FA4 (though we haven't checked cocalc or nteract)

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?

@jasongrout
Copy link
Member

Also, FA5 includes a shim css file that maps v4 names to v5, so apparently it doesn't have to be backwards incompatible.

@martinRenou
Copy link
Member

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.

@martinRenou
Copy link
Member

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 :)

@jasongrout
Copy link
Member

Closing as resolved. Upgrading the notebook can be a ticket for that repo. Thanks again!

@lock lock bot added the resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label May 20, 2020
@lock lock bot locked as resolved and limited conversation to collaborators May 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Projects
None yet
Development

No branches or pull requests

4 participants