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

Add a default "showitem" configuration #23

Closed
davidsteeb opened this issue Jul 11, 2020 · 3 comments · Fixed by #29
Closed

Add a default "showitem" configuration #23

davidsteeb opened this issue Jul 11, 2020 · 3 comments · Fixed by #29

Comments

@davidsteeb
Copy link
Contributor

It would be nice to add a default container showitem configuration for container elements that do not provide a more expansive showitem configuration.

When registering a new container this "showitem" TCA configuration could be added; as long as there is no special configuration set up following the registration of the container this default setup will be used. It would make sense to add default fields (like CType, language, access) to a default configuration so the container is working in most instances.

@achimfritz
Copy link
Contributor

i would prefer "sys_language_uid,CType,tx_container_parent,colPos,hidden" as default configuration (tx_container_parent is required to place the container into an other container, cType colPos is required to place the container into an BE-Layout column) (? or?)

@ervaude
Copy link
Contributor

ervaude commented Jul 13, 2020

Resolved by #29

@ervaude ervaude closed this as completed Jul 13, 2020
@ervaude
Copy link
Contributor

ervaude commented Jul 13, 2020

Wasnt merged yet :)

@ervaude ervaude reopened this Jul 13, 2020
@bmack bmack closed this as completed in #29 Jul 13, 2020
bmack added a commit that referenced this issue Jul 13, 2020
[FEATURE] add a default showitem configuration to container #23
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 a pull request may close this issue.

3 participants