NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
18 Jul 2021

This pr adds spacing between the icon and the text. as far as I can see this is only used in one place as shown below

Before

image

After

image

This is a css change

avatar brianteeman brianteeman - open - 18 Jul 2021
avatar brianteeman brianteeman - change - 18 Jul 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Jul 2021
Category Administration Templates (admin) NPM Change
avatar ChristineWk ChristineWk - test_item - 18 Jul 2021 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 18 Jul 2021

I have tested this item successfully on 4927fa3


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

avatar richard67 richard67 - test_item - 18 Jul 2021 - Tested successfully
avatar richard67
richard67 - comment - 18 Jul 2021

I have tested this item successfully on 4927fa3

Tested LTR and RTL.


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

avatar richard67 richard67 - change - 18 Jul 2021
Status Pending Ready to Commit
Labels Added: ? NPM Resource Changed
avatar richard67
richard67 - comment - 18 Jul 2021

RTC


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

avatar Quy
Quy - comment - 18 Jul 2021

Under Privacy Dashboard, the icon is further apart from the text.

34815

avatar richard67
richard67 - comment - 18 Jul 2021

Under Privacy Dashboard, the icon is further apart from the text.

@Quy Without this PR it looks like this:

2021-07-18_pr-34815

In both cases the distance between the icon and its text seems to be the same for both kinds of icons, but the icons have a different width and so it looks not well aligned in both cases.

So it seems the issue is not caused by this PR, it just becomes a bit more visible.

I think a fa-fw class in the markup could fix that.

@brianteeman Do you want to make a PR for that, too?

avatar chmst chmst - change - 20 Jul 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-07-20 19:46:54
Closed_By chmst
Labels Added: ?
avatar chmst chmst - close - 20 Jul 2021
avatar chmst chmst - merge - 20 Jul 2021
avatar chmst
chmst - comment - 20 Jul 2021

Merged - a little bit more space is better than no space. Thanks.

avatar brianteeman
brianteeman - comment - 20 Jul 2021

thanks

Add a Comment

Login with GitHub to post a comment