NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
20 Apr 2019

This is quite a big change and will need a composer and npm install to test

Many of the icons have changed name with FA5 see https://fontawesome.com/how-to-use/on-the-web/setup/upgrading-from-version-4#name-changes

The icomoon mapping has been updated with the name changes

Where possible I have updated any icons in core to the new names but I may have missed some.

avatar brianteeman brianteeman - open - 20 Apr 2019
avatar brianteeman brianteeman - change - 20 Apr 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Apr 2019
Category Administration com_associations com_banners com_categories com_contact com_content com_cpanel com_fields com_languages com_media NPM Change com_menus com_modules com_newsfeeds com_plugins com_redirect com_tags com_templates
avatar brianteeman brianteeman - change - 20 Apr 2019
The description was changed
avatar brianteeman brianteeman - edited - 20 Apr 2019
avatar brianteeman brianteeman - change - 20 Apr 2019
Labels Added: NPM Resource Changed ?
avatar brianteeman brianteeman - change - 20 Apr 2019
The description was changed
avatar brianteeman brianteeman - edited - 20 Apr 2019
avatar Hackwar
Hackwar - comment - 20 Apr 2019

Thank you for doing all this work. There is a CSS to keep on using FA4 names with FA5. Should we use this to keep the changes smaller? I also really, REALLY don't like that we can't use the Joomla icon in our own CMS from that icon font. Should we contact them and ask them to move it to the free group?

avatar mbabker
mbabker - comment - 20 Apr 2019

I also really, REALLY don't like that we can't use the Joomla icon in our own CMS from that icon font.

Umm, why? I updated https://github.com/joomla/framework.joomla.org to use FA5 a while ago and use the Joomla icon there without issue, you just have to make sure you're loading the brand icons (none of which are in the pro group).

avatar brianteeman
brianteeman - comment - 20 Apr 2019

I must have misread the terms of the brand file.

As for the v4 to v5 shim @wilsonge correctly pointed out that joomla has never shipped a release with fa4

avatar brianteeman
brianteeman - comment - 20 Apr 2019

I will fix to include the brand if requested but for the sake of one icon I would prefer a different solution with less weight

avatar wilsonge
wilsonge - comment - 20 Apr 2019

For the Joomla brand icon let's create our own class that loads the logo.svg in the template https://github.com/joomla/joomla-cms/blob/4.0-dev/administrator/templates/atum/images/logo.svg

avatar brianteeman
brianteeman - comment - 20 Apr 2019

yes that's what i was thinking. that should be done in its own pr

avatar brianteeman brianteeman - change - 20 Apr 2019
The description was changed
avatar brianteeman brianteeman - edited - 20 Apr 2019
avatar wilsonge
wilsonge - comment - 20 Apr 2019

The other option thinking about it is to load the brand icons in the backend template only. For things like the social media classes might be useful.

avatar brianteeman
brianteeman - comment - 20 Apr 2019

if the brand icons are included then it should be in both

avatar wilsonge
wilsonge - comment - 20 Apr 2019

Personally I don't think that's right. We can be more tight with the resources loaded in the frontend than the backend

avatar brianteeman
brianteeman - comment - 20 Apr 2019

The use case for social media icons its much greater in the front end

avatar mbabker
mbabker - comment - 20 Apr 2019

If anything it's more common to load brand icons in the frontend than it is in the backend.

avatar dgrammatiko
dgrammatiko - comment - 21 Apr 2019

@wilsonge according to your decision ( joomla/40-backend-template#441 (comment) ) the front end shouldn't have a dependency on FA. I hope you didn't change your mind on this.

PS. Also for this case to stand true all of the form fields should be font awesome free, as the fields are common to both sides frontend -backend...

avatar brianteeman
brianteeman - comment - 21 Apr 2019

I have added brands and updated the markup where necessary to use it (fa => fab)
It is a separate commit to can easily be removed

avatar richard67
richard67 - comment - 21 Apr 2019

@brianteeman Shouldn't there be a change "fa" => "fas" for those icons in the standard package, e.g. change "fa fa-pen-square" to "fas fa-pen-square"? See https://fontawesome.com/icons/pen-square?style=solid

That means for Fontawesome 5, the "fa" is in general changed to a three letter class where the last letter is the subpackage where it belongs to, i.e. "b" for brand, "s" for solid, "l" for light and "r" for regular.

avatar brianteeman
brianteeman - comment - 21 Apr 2019

no there is no need for that. FA5 supports both fa and fas

avatar richard67
richard67 - comment - 21 Apr 2019

Hmm, their icon pages don't show that.

avatar richard67
richard67 - comment - 21 Apr 2019

I see: "fas or fa = Font Awesome Solid". Thanks for the info. Seems I missed that in past.

avatar brianteeman
brianteeman - comment - 21 Apr 2019

@richard67 and of course I followed best practice and checked that it worked before I submitted the PR

avatar richard67
richard67 - comment - 21 Apr 2019

@brianteeman I never assumed something else. But as we are all humans only, we all can make mistakes ;-)

avatar wilsonge wilsonge - change - 21 Apr 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-04-21 11:48:05
Closed_By wilsonge
avatar wilsonge wilsonge - close - 21 Apr 2019
avatar wilsonge wilsonge - merge - 21 Apr 2019
avatar wilsonge
wilsonge - comment - 21 Apr 2019

That'll do for now :) We can definitely work on how we optimise things in frontend. But this is now just a straight version upgrade so we don't have to worry about that here

Thanks!

avatar brianteeman
brianteeman - comment - 21 Apr 2019

Thanks - learnt a lot writing this pr

avatar Scrabble96
Scrabble96 - comment - 17 Aug 2019

It's a shame, but fontawesome 4.7 fonts like file-o, which used to be white/transparent with a colour outline, have now been classed as 'light' in v 5.0 - and are now Pro, so no longer free.

In Joomla 3x, in the template editor directory tree, for example, the file icon is light and easily distinguishable at a glance from from folders:

J3-white-icons

In Joomla 4.0 dev they are all solid:
j4-solid-icons

See pics of the different fontawesome options:

4.7 free:
fontawesome-4 7-file-o

5.0 free
fontawesome-5 0-free-file

5.0 Pro
fontawesome-5 0-pro-file

Is there a case for keeping both versions?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Aug 2019

@Scrabble96 can you please open an new Issue (plus link to this) as Comments on closed one didn't get much notice.

avatar Scrabble96
Scrabble96 - comment - 17 Aug 2019

@Scrabble96 can you please open an new Issue (plus link to this) as Comments on closed one didn't get much notice.

Hi, @franz-wohlkoenig. Thanks, I have just done so on issue #25915.

Add a Comment

Login with GitHub to post a comment