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

[4.0] Change fa-fw to icon-fw #33980

Merged
merged 1 commit into from
May 20, 2021
Merged

[4.0] Change fa-fw to icon-fw #33980

merged 1 commit into from
May 20, 2021

Conversation

Quy
Copy link
Contributor

@Quy Quy commented May 19, 2021

Summary of Changes

To remove references to FontAwesome, use generic icon class.
Change fa-fw to icon-fw.
Change order for consistency.

Testing Instructions

Code review.

or

Log in to the administration area.
View Post Installation Messages/Private Messages buttons in the upper right corner.
Icons same before/after PR.

@ChristineWk
Copy link

ChristineWk commented May 19, 2021

@Quy Sorry, maybe I make something wrong.
Hv now 50:50 :-)
View Post Installation Messages = OK
screen shot 2021-05-19 at 17 24 40

<span class="icon-bell icon-fw" aria-hidden="true"></span>

But I couldn't see the Private Messages buttons in the upper right corner.


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

@Quy
Copy link
Contributor Author

Quy commented May 19, 2021

But I couldn't see the Private Messages buttons in the upper right corner.

It is hidden by default when there are no new private messages.

To make it appear, please do the following in this file:
administrator/modules/mod_messages/tmpl/default.php

if ($hideLinks || $countUnread < 1)
to
if ($hideLinks)

@ChristineWk
Copy link

I have tested this item ✅ successfully on d24022c

@Quy Thanks for the hint. Changed the code via FTP accordingly.


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

@ChristineWk
Copy link

ChristineWk commented May 19, 2021

OK now:

<span class="icon-envelope icon-fw" aria-hidden="true"></span>
screen shot 2021-05-19 at 17 54 28


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

@brianteeman
Copy link
Contributor

I trust if you are going to insist on this pointless change then you will be updating all the other instances in the admin. There is a valid reason to use a generic class in the frontend but none in the admin

@Quy
Copy link
Contributor Author

Quy commented May 19, 2021

Nope. There are only 2 instances of fa- in the backend, thus, this PR. The rest are icon-.

@brianteeman
Copy link
Contributor

Search again

@Quy
Copy link
Contributor Author

Quy commented May 19, 2021

Well this PR makes these icons consistent with the other icons in the header. I am blind to what you are referring to so please enlighten me.

@richard67
Copy link
Member

I have tested this item ✅ successfully on d24022c


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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 20, 2021
@richard67 richard67 added this to the Joomla 4.0 milestone May 20, 2021
@brianteeman
Copy link
Contributor

sorry I was searching a branch of mine that had a lot of extensions installed and I didnt realise. There is only one fa- remaining

fa-8x

@Quy
Copy link
Contributor Author

Quy commented May 20, 2021

Mystery solved. I will do a PR for fa-8x. Thank you!

@rdeutz rdeutz merged commit a2d5460 into joomla:4.0-dev May 20, 2021
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 20, 2021
@Quy Quy deleted the fa-fw branch December 1, 2021 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants