? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
15 Apr 2020

Redo of #27344 to fix conflicts and do incremental PRs for easy testing.

Thanks @chang-zhao for the initial PR.

Summary of Changes

Replace width inline styles with classes.

w-25 and w-50 are Bootstrap sizing classes so no need to create these classes.

Testing Instructions

Code review.
or
Log in to administration.
No visual changes in the following:
See Logged-in Users module.
See System > System Information > System Information tab

avatar Quy Quy - open - 15 Apr 2020
avatar Quy Quy - change - 15 Apr 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Apr 2020
Category Administration com_admin com_content com_contenthistory com_templates com_users Modules
avatar ciar4n
ciar4n - comment - 15 Apr 2020

What is the benefit of replacing style width with classes?

avatar Quy
Quy - comment - 15 Apr 2020

To address CSP. See #27297.

avatar richard67
richard67 - comment - 15 Apr 2020

@Quy I think on your description you meant „... no need to create these classes“, but the “no” is missing.

avatar Quy Quy - change - 15 Apr 2020
The description was changed
avatar Quy Quy - edited - 15 Apr 2020
avatar Quy
Quy - comment - 15 Apr 2020

Fixed. Thanks.

avatar jwaisner
jwaisner - comment - 16 Apr 2020

I have tested this item successfully on 9fa541f


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

avatar jwaisner jwaisner - test_item - 16 Apr 2020 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 16 Apr 2020

I have tested this item successfully on 9fa541f


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

avatar ChristineWk ChristineWk - test_item - 16 Apr 2020 - Tested successfully
avatar Quy Quy - change - 16 Apr 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 16 Apr 2020

RTC


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

avatar HLeithner HLeithner - change - 17 Apr 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-04-17 06:26:49
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 17 Apr 2020
avatar HLeithner HLeithner - merge - 17 Apr 2020
avatar HLeithner
HLeithner - comment - 17 Apr 2020

Thanks

Add a Comment

Login with GitHub to post a comment