? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
24 Nov 2019

Pull Request for Issue #27136.

Summary of Changes

This PR creates some column-count classes and applies them to the joomla.edit.params layout.

Basically means that fields get ordered vertically (J3) instead of horizontally which is the case when using grid.

If this gets merged then I will apply on all other forms.

Testing Instructions

Apply this patch and run node build.js --compile-css for updating the changed SCSS. Alternatively, you can run npm i.

Navigate to article edit -> Options tab

Before PR

o4grXhrkUc

WIth PR

iydq12lHd0

Documentation Changes Required

avatar ciar4n ciar4n - open - 24 Nov 2019
avatar ciar4n ciar4n - change - 24 Nov 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Nov 2019
Category Administration Templates (admin) Layout
avatar brianteeman
brianteeman - comment - 24 Nov 2019

THANK YOU

avatar Quy
Quy - comment - 24 Nov 2019

Go to Content >Categories
Edit Uncategorised
Go to Options
Select is in the next column

27141

avatar ciar4n ciar4n - change - 24 Nov 2019
The description was changed
avatar ciar4n ciar4n - edited - 24 Nov 2019
avatar ciar4n ciar4n - change - 24 Nov 2019
Labels Added: ?
avatar ciar4n
ciar4n - comment - 24 Nov 2019

Thx @Quy. That should be ok now.

avatar brianteeman brianteeman - test_item - 24 Nov 2019 - Tested successfully
avatar brianteeman
brianteeman - comment - 24 Nov 2019

I have tested this item successfully on c746736


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

avatar Quy Quy - test_item - 24 Nov 2019 - Tested successfully
avatar Quy
Quy - comment - 24 Nov 2019

I have tested this item successfully on c746736


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

avatar Quy Quy - change - 24 Nov 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 24 Nov 2019

RTC


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

avatar Quy Quy - change - 24 Nov 2019
Status Ready to Commit Pending
avatar Quy
Quy - comment - 24 Nov 2019

Administrator Template Styles > autum > Image Settings

27141a

avatar ciar4n
ciar4n - comment - 24 Nov 2019

@Quy Should be ok now. XML update so no need to recompile css.

avatar Quy
Quy - comment - 24 Nov 2019

If this gets merged then I will apply on all other forms.

Is this related to my last comment?

The before and after look differently. Is this the expected behavior?

27141-before

27141-after

avatar ciar4n
ciar4n - comment - 25 Nov 2019

Is this related to my last comment?

There is another 60+ files that use the current grid based forms. A follow-up PR would apply the changes here across all such files and remove the old grid based forms CSS. The Atum XML from your last comment was created for the grid based forms so the field order was different. Most XMLs have been created with the column based forms used in Joomla 3 and therefore should be better suited to the changes here.

The before and after look differently. Is this the expected behavior?

Yes. There will be differences. Currently, forms are placed in a grid so forms are inline with each other both vertically and horizontally. This PR changes forms to a column layout.

Grid layout...

image

Column layout...

image

Look at any multi-column form outside of Joomla and they will always be in a column layout and read vertically, never a grid.

avatar wilsonge wilsonge - change - 25 Nov 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-11-25 11:08:24
Closed_By wilsonge
avatar wilsonge wilsonge - close - 25 Nov 2019
avatar wilsonge wilsonge - merge - 25 Nov 2019
avatar wilsonge
wilsonge - comment - 25 Nov 2019

Thanks @ciar4n happy to move onto doing this elsewhere

avatar ciar4n
ciar4n - comment - 25 Nov 2019

Thank you for the tests.

avatar ciar4n
ciar4n - comment - 17 Jan 2020

To complete I need some tests on #27141. Otherwise, this should be reverted.

avatar wilsonge
wilsonge - comment - 18 Jan 2020

That got merged two months ago?

avatar ciar4n
ciar4n - comment - 18 Jan 2020

Ah sorry .. wrong pr... this one.. #27153

Add a Comment

Login with GitHub to post a comment