Task #82706

Fieldset labels missing in mails

Added by Mathias Brodala over 2 years ago. Updated about 1 month ago.

Status:
Closed
Priority:
Should have
Category:
Form Framework
Target version:
-
Start date:
2017-10-09
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Sprint Focus:
Remote Sprint

Description

Currently Fieldset elements are completely skipped in the default template for the EmailFinisher. If one e.g. has multiple sections in his form with similar fields like street, city, etc. the mail currently is hard to read since there is no headline or similar separating the fields.

This could be achieved by rendering the fieldset labels in forms. Currently this can be achieved only with a custom mail template and globally setting renderingOptions._isCompositeFormElement: false for Fieldset in the setup. Otherwise fieldset elements are completely skipped by the RenderAllFormValuesViewHelper


Related issues

Related to TYPO3 Core - Feature #84133: Variants - Frontend implementation Closed 2018-05-24

Associated revisions

Revision 03f537db (diff)
Added by Mathias Brodala 2 months ago

[FEATURE] Render fieldset labels in form templates

This adds fieldset labels as separators on the summary page and mails.

Resolves: #82706
Releases: master
Change-Id: I09ab461cf7efa49a7658b950b30a7a5f3b73282d
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389
Tested-by: TYPO3com <>
Tested-by: Susanne Moog <>
Tested-by: Daniel Goerz <>
Reviewed-by: Susanne Moog <>
Reviewed-by: Daniel Goerz <>

History

#1 Updated by Mathias Brodala over 2 years ago

This could be achieved by rendering the fieldset labels in forms.

... in mails, of course.

#2 Updated by Ralf Zimmermann almost 2 years ago

#3 Updated by Ralf Zimmermann almost 2 years ago

  • Status changed from New to Accepted

If #84133 is done, we can handle this issue with variants.
I will provide a patchset after #84133.

#4 Updated by Bjoern Jacob 10 months ago

  • Sprint Focus set to Remote Sprint

#5 Updated by Bjoern Jacob 9 months ago

  • Tracker changed from Bug to Task
  • Sprint Focus deleted (Remote Sprint)

#6 Updated by Bjoern Jacob 2 months ago

  • Sprint Focus set to Remote Sprint

#7 Updated by Gerrit Code Review 2 months ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389

#8 Updated by Gerrit Code Review 2 months ago

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389

#9 Updated by Gerrit Code Review 2 months ago

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389

#10 Updated by Gerrit Code Review 2 months ago

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389

#11 Updated by Mathias Brodala 2 months ago

  • Assignee set to Mathias Brodala

#12 Updated by Gerrit Code Review 2 months ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389

#13 Updated by Gerrit Code Review 2 months ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389

#14 Updated by Gerrit Code Review 2 months ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389

#15 Updated by Gerrit Code Review 2 months ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389

#16 Updated by Gerrit Code Review 2 months ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62389

#17 Updated by Mathias Brodala 2 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#18 Updated by Benni Mack about 1 month ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF