Project

General

Profile

Actions

Bug #102291

closed

Add fluidAdditionalAttributes to Fieldset elements

Added by Henrik Ziegenhain about 1 year ago. Updated about 1 month ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Form Framework
Start date:
2023-10-30
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
11
PHP Version:
8.1
Tags:
Complexity:
no-brainer
Is Regression:
Sprint Focus:
On Location Sprint

Description

Currently it is not possible to pass additionalAttributes to "Fieldset" form elements.
For (nearly?) all other element it is possible.

In typo3/sysext/form/Resources/Private/Frontend/Partials/Fieldset.html just replace

<fieldset id="{element.uniqueIdentifier}" class="{element.properties.elementClassAttribute}">

with

    <f:variable name="additionalAttributes" value="" />
    <f:for as="value" key="key" each="{formvh:translateElementProperty(element: element, property: 'fluidAdditionalAttributes')}">
        <f:variable name="additionalAttributes" value='{additionalAttributes} {key}="{value}"' />
    </f:for>
    <fieldset id="{element.uniqueIdentifier}" class="{element.properties.elementClassAttribute}" {additionalAttributes -> f:format.raw()}>

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #104612: Error on form "Fieldset" without "fluidAdditionalAttributes"ClosedMathias Brodala2024-08-13

Actions
Actions #1

Updated by Gerrit Code Review about 1 year ago

  • Status changed from New to Under Review

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

Actions #2

Updated by Gerrit Code Review about 1 year ago

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

Actions #3

Updated by Georg Ringer 4 months ago

  • Sprint Focus set to On Location Sprint
Actions #4

Updated by Gerrit Code Review 4 months ago

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

Actions #5

Updated by Gerrit Code Review 4 months ago

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

Actions #6

Updated by Gerrit Code Review 3 months ago

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

Actions #7

Updated by Anonymous 3 months ago

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

Updated by Mathias Brodala 3 months ago

  • Related to Bug #104612: Error on form "Fieldset" without "fluidAdditionalAttributes" added
Actions #9

Updated by Mathias Brodala 3 months ago

Notice that this leads to an error if no fluidAdditionalAttributes are set (which would be the default), see #104612

Actions #10

Updated by Benni Mack about 1 month ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF