User tests: Successful: Unsuccessful:
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.
Status | New | ⇒ | Pending |
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 |
Labels |
Added:
NPM Resource Changed
?
|
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).
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
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
yes that's what i was thinking. that should be done in its own pr
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.
if the brand icons are included then it should be in both
Personally I don't think that's right. We can be more tight with the resources loaded in the frontend than the backend
The use case for social media icons its much greater in the front end
If anything it's more common to load brand icons in the frontend than it is in the backend.
@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...
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
@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.
no there is no need for that. FA5 supports both fa and fas
Hmm, their icon pages don't show that.
I see: "fas or fa = Font Awesome Solid". Thanks for the info. Seems I missed that in past.
@richard67 and of course I followed best practice and checked that it worked before I submitted the PR
@brianteeman I never assumed something else. But as we are all humans only, we all can make mistakes ;-)
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-04-21 11:48:05 |
Closed_By | ⇒ | wilsonge |
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!
Thanks - learnt a lot writing this pr
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:
In Joomla 4.0 dev they are all solid:
See pics of the different fontawesome options:
Is there a case for keeping both versions?
@Scrabble96 can you please open an new Issue (plus link to this) as Comments on closed one didn't get much notice.
@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.
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?