? Failure

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
6 Jun 2017

Pull Request for Issue # .

Summary of Changes

Gives the top right header icons equal widths.

Before

header-icons1

After

header-icons2

avatar ciar4n ciar4n - open - 6 Jun 2017
avatar ciar4n ciar4n - change - 6 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jun 2017
Category Administration Templates (admin)
avatar C-Lodder
C-Lodder - comment - 7 Jun 2017

If it's just the icons themselves that needs to be fixed with, then you can use the fa-fw class, e.g:

fa fa-external-link fa-fw

Would be a nicer approach IMO

avatar ciar4n ciar4n - change - 7 Jun 2017
Labels Added: ?
avatar ciar4n
ciar4n - comment - 7 Jun 2017

If it's just the icons themselves that needs to be fixed with, then you can use the fa-fw class, e.g:

Won't work as it would be overruled by https://github.com/joomla/joomla-cms/blob/4.0-dev/administrator/templates/atum/scss/blocks/_header.scss#L68

Inline with what you suggest I have moved the width to the icon level

avatar brianteeman brianteeman - change - 8 Jun 2017
Milestone Added:
avatar brianteeman brianteeman - change - 8 Jun 2017
Milestone Added:
avatar N6REJ
N6REJ - comment - 18 Jun 2017

what is a "fa" class and where is it documented?

avatar ciar4n
ciar4n - comment - 18 Jun 2017

fa is a FontAwesome class which is the icon library been used for Joomla 4... http://fontawesome.io/

avatar C-Lodder
C-Lodder - comment - 18 Jun 2017

@N6REJ - Just FYI you can still use the same icon-xxx classes as you can in J3. We've mapped all the classes to it's B/C

avatar wilsonge wilsonge - change - 18 Jun 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-06-18 14:28:31
Closed_By wilsonge
avatar wilsonge wilsonge - close - 18 Jun 2017
avatar wilsonge wilsonge - merge - 18 Jun 2017

Add a Comment

Login with GitHub to post a comment