? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
25 Jun 2019

Render all the fields in the sidebar using the layout. There should be no visible difference or any difference in the generated code.

avatar brianteeman brianteeman - open - 25 Jun 2019
avatar brianteeman brianteeman - change - 25 Jun 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Jun 2019
Category Administration com_plugins
avatar richard67 richard67 - test_item - 25 Jun 2019 - Tested successfully
avatar richard67
richard67 - comment - 25 Jun 2019

I have tested this item successfully on 21b5268

No visual changes. Code shows differences but they are expected, e.g. <div class="form-vertical form-no-margin"> missing after the patch because removed with that PR. The code of the fields themselves is same as without the patch. @brianteeman maybe you should slighty modify the testing instructions.


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

avatar brianteeman
brianteeman - comment - 25 Jun 2019

The div with the class is not needed as the classes are already present on the fieldset

avatar richard67
richard67 - comment - 25 Jun 2019

Yes, I saw. Maybe you just should have put the word "generated" in bold letters in "difference in the generated code" for fast readers like me ;-)

avatar Quy Quy - test_item - 26 Jun 2019 - Tested successfully
avatar Quy
Quy - comment - 26 Jun 2019

I have tested this item successfully on 21b5268


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

avatar Quy
Quy - comment - 26 Jun 2019

I have tested this item successfully on 21b5268


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

avatar Quy Quy - change - 26 Jun 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 26 Jun 2019

RTC


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

avatar Quy
Quy - comment - 26 Jun 2019

RTC


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

avatar wilsonge wilsonge - close - 27 Jun 2019
avatar wilsonge wilsonge - merge - 27 Jun 2019
avatar wilsonge wilsonge - change - 27 Jun 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-06-27 10:48:35
Closed_By wilsonge
Labels Added: ? ?
avatar wilsonge
wilsonge - comment - 27 Jun 2019

Thanks!

avatar brianteeman
brianteeman - comment - 27 Jun 2019

Thanks

Add a Comment

Login with GitHub to post a comment