? Documentation Required Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
20 Jun 2021

The entire point of the _icomoon.scss file is to map fontawesome icons to the icons used in joomla 3 from icomoon

Somehow the wrong icons were used for icon-address and icon-vcard

You can see the original icomoon icons here https://docs.joomla.org/J3.x:Joomla_Standard_Icomoon_Fonts

avatar brianteeman brianteeman - open - 20 Jun 2021
avatar brianteeman brianteeman - change - 20 Jun 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Jun 2021
Category Repository NPM Change
avatar Quy Quy - test_item - 22 Jun 2021 - Tested successfully
avatar Quy
Quy - comment - 22 Jun 2021

I have tested this item successfully on 06b2784


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34576.

avatar RickR2H
RickR2H - comment - 23 Jun 2021

Testing the single contact I noticed that the address icon is wrong.
Change in: components\com_contact\tmpl\contact\default_address.php on line: 25 change "icon-address" to "icon-address-book"

avatar brianteeman
brianteeman - comment - 23 Jun 2021

With this PR you dont need to make that change - thats the whole point of it ;)

avatar RickR2H
RickR2H - comment - 23 Jun 2021

With this PR you dont need to make that change - thats the whole point of it ;)

Okay! I see what you mean, I think there is something wrong with my SCSS compile. I rebuild the SCSS but the old icon remains. I will do an other clean test just to get it to work on my end.

avatar RickR2H
RickR2H - comment - 23 Jun 2021

Hey Brian,

I did a clean install to test the icons. I applied the patch and compiled the SCSS again. Still I get the wrong icon. Am I missing something?
Image-5

avatar brianteeman
brianteeman - comment - 23 Jun 2021

Sorry you have to rebuild everything with npm ci to get the correct icon

avatar RickR2H RickR2H - test_item - 23 Jun 2021 - Tested successfully
avatar RickR2H
RickR2H - comment - 23 Jun 2021

I have tested this item successfully on 06b2784

After rebuilding I got the correct result!


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34576.

avatar chmst chmst - change - 24 Jun 2021
Status Pending Ready to Commit
avatar chmst
chmst - comment - 24 Jun 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34576.

avatar wilsonge wilsonge - change - 27 Jun 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-06-27 10:22:47
Closed_By wilsonge
Labels Added: ? Documentation Required
avatar wilsonge wilsonge - close - 27 Jun 2021
avatar wilsonge wilsonge - merge - 27 Jun 2021
avatar wilsonge
wilsonge - comment - 27 Jun 2021

Thanks!

avatar brianteeman
brianteeman - comment - 27 Jun 2021

thanks

Add a Comment

Login with GitHub to post a comment