Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.0] Nested fieldsets #29398

Closed
wants to merge 1 commit into from
Closed

Conversation

brianteeman
Copy link
Contributor

This PR stops splitting nested fieldsets using the params layout from being split again into columns

Possible solution to #29388 and #29254

before

image

after

image

I have not checked to see if there are other places where this happens other than the example in the screenshot with com_tags

This PR stops splitting nested fieldsets using the params layout from being split again into columns
@Quy
Copy link
Contributor

Quy commented Jun 2, 2020

See similar PR #28790

@brianteeman
Copy link
Contributor Author

@Quy that one still splits into columns and would still cause the problems reported elsewhere

@Quy
Copy link
Contributor

Quy commented Jun 2, 2020

Yes, then we run into issue #27672 with one column.

@Quy
Copy link
Contributor

Quy commented Jun 2, 2020

See George's comment: #28790 (comment)

@brianteeman
Copy link
Contributor Author

Not with this pr

@Quy
Copy link
Contributor

Quy commented Jun 2, 2020

OK. Let me test it later and report back.

@Quy
Copy link
Contributor

Quy commented Jun 3, 2020

Before PR:
29398-before

After PR:
29398-after

So #27672 and George's comment #28790 (comment) are related.

@brianteeman
Copy link
Contributor Author

Then my code is wrong. The intention was that it would only stop nested fieldsets from splitting in to columns but keep the columns on the non-nested fieldsets

@Quy
Copy link
Contributor

Quy commented Jun 3, 2020

Similar changes have to be done as #28790 but for one column. See Edit article > Configure Edit Screen where it is 2 columns and should be 1 for consistency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants