Skip to content

Conversation

@billdagou
Copy link
Contributor

@billdagou billdagou commented Sep 11, 2025

Fluid used the XHTML syntax to represent boolean values. This change adjusts
the TagBuilder to use the modern syntax. This change is considered non-breaking
because it results in the same interpretation of the browser. However, it will
not be backported to Fluid v4 to be safe.

Before:

<my:tagBased required="{true}" />
Result: <input required="required" />

After:

<my:tagBased required="{true}" />
Result: <input required />

@s2b s2b changed the title Non-breaking task for adjusted markup for true and false [TASK] Use modern HTML syntax for boolean attributes Sep 15, 2025
@s2b
Copy link
Contributor

s2b commented Sep 15, 2025

What was the reason for the additional requireAttributeValues configuration? From my point of view, this isn't necessary, we can just switch to the alternative syntax.

@s2b s2b marked this pull request as draft September 15, 2025 09:43
@billdagou
Copy link
Contributor Author

billdagou commented Sep 16, 2025

To switch between <input required /> and <input required="required" />, like for config.doctype = xhtml_* or config.doctype = html5 in TYPO3.

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.

2 participants