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

[RTM] Display form fieldsets as wrappers in the backend view. #1102

Merged
merged 4 commits into from
Nov 24, 2017

Conversation

dmolineus
Copy link
Contributor

@dmolineus dmolineus commented Sep 25, 2017

Wrapper elements provides a cleaner view of complex elements in the parent view. As the feature is not limited to CEs it would be nice to use it for form fieldsets as well.

  • Split fieldset into start and stop element
  • Update language files
  • Provide migration script

If this PR is accepted I'll provide the missing tasks as well.

@fritzmg
Copy link
Contributor

fritzmg commented Sep 25, 2017

Oh yes finally 😁

@dmolineus
Copy link
Contributor Author

dmolineus commented Sep 25, 2017

Migration script provided with contao/installation-bundle#69

@dmolineus dmolineus changed the title [WIP] Display form fieldsets as wrappers in the backend view. [RFC] Display form fieldsets as wrappers in the backend view. Sep 25, 2017
@fritzmg
Copy link
Contributor

fritzmg commented Sep 25, 2017

This will break some extensions though, e.g. terminal42/contao-conditionalformfields

@Toflar
Copy link
Member

Toflar commented Sep 26, 2017

It's not API and I think it won't be really a problem to adjust the extension to it ;)

@fritzmg
Copy link
Contributor

fritzmg commented Sep 26, 2017

I know, just saying ;)

@leofeyer leofeyer force-pushed the develop branch 3 times, most recently from e37e989 to dad087d Compare October 3, 2017 19:03
@dmolineus dmolineus changed the title [RFC] Display form fieldsets as wrappers in the backend view. [RTM] Display form fieldsets as wrappers in the backend view. Oct 5, 2017
@leofeyer leofeyer force-pushed the develop branch 12 times, most recently from 4c8802b to 4f5e7b2 Compare November 2, 2017 22:53
@leofeyer leofeyer force-pushed the develop branch 2 times, most recently from 495cb11 to 1358688 Compare November 5, 2017 16:21
@leofeyer leofeyer force-pushed the develop branch 3 times, most recently from da44f69 to 90840d3 Compare November 20, 2017 13:28
@leofeyer leofeyer added this to the 4.5.0 milestone Nov 22, 2017
@ghost ghost closed this Nov 22, 2017
@contao contao deleted a comment Nov 22, 2017
@leofeyer leofeyer reopened this Nov 22, 2017
@leofeyer leofeyer merged commit 2e37b40 into contao:develop Nov 24, 2017
@leofeyer
Copy link
Member

Thank you @dmolineus.

@dmolineus dmolineus deleted the feature/fieldset-wrappers branch November 24, 2017 08:56
leofeyer pushed a commit that referenced this pull request Dec 16, 2019
…(see #1102)

Description
-----------

According to the phpdoc comment, `Response::getContent()` may also return `false`.

Commits
-------

ce71f642 Make sure we have the correct type when a search document is created
0758a2af Merge branch 'master' into bugfix/fix-typehint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants