? ? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
10 May 2020

Continuation of PR #28692

Summary of Changes

Add w- classes.
Replace inline style widths % with w-.

Testing Instructions

Code review.

avatar Quy Quy - open - 10 May 2020
avatar Quy Quy - change - 10 May 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 May 2020
Category Administration com_banners com_categories com_content com_templates com_users com_workflow Templates (admin)
avatar Quy Quy - change - 10 May 2020
Title
Replace inline style width 3%, 6%, 7% with w-3, w-6, w-7
[4.0] Replace inline style width 3%, 6%, 7% with w-3, w-6, w-7
avatar Quy Quy - edited - 10 May 2020
avatar ChristineWk ChristineWk - test_item - 10 May 2020 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 10 May 2020

I have tested this item successfully on 2b79b2e


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

avatar richard67 richard67 - test_item - 11 May 2020 - Tested successfully
avatar richard67
richard67 - comment - 11 May 2020

I have tested this item successfully on 2b79b2e


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

avatar richard67 richard67 - change - 11 May 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 11 May 2020

RTC


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

avatar richard67 richard67 - change - 11 May 2020
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 11 May 2020

I understand and agree with the addition of the larger numbers - but this is really going too far. Why stop at w-3, w-6, w-7 whats wrong with w-4, w-8 and w-9

avatar Quy
Quy - comment - 11 May 2020

I am just going through the existing values. There is one more round to do. Then these can be adjusted/rounded up if desired to reduce the number of sizes.

avatar Quy
Quy - comment - 12 May 2020

I am going to keep it as it. The only difference is that it is now class and not style.

avatar infograf768 infograf768 - change - 13 May 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-05-13 05:18:42
Closed_By infograf768
Labels
avatar infograf768 infograf768 - close - 13 May 2020
avatar infograf768 infograf768 - merge - 13 May 2020
avatar infograf768
infograf768 - comment - 13 May 2020

tks

Add a Comment

Login with GitHub to post a comment