Bug #48121
TextareaViewHelper is missing valid html5 argument placeholder ...
| Status: | Resolved | Start date: | 2013-05-12 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | - | % Done: | 100% |
|
| Category: | ViewHelpers | |||
| Target version: | TYPO3 Flow Base Distribution - 2.0 | |||
| Has patch: | Yes | Affected Flow version: | Flow 2.0.0 beta 2 |
Description
TextareaViewHelper is missing valid html5 argument placeholder ...
Using
<f:form.textfield id="powermail_field_{field.marker}" placeholder="{field.title -> vh:string.RawAndRemoveXss()}" name="field[{field.uid}]" value="{vh:Misc.PrefillField(field: '{field}')}" class="powermail_field powermail_input {vh:Misc.ValidationClass(field: '{field}')} {vh:Misc.ErrorClass(field: '{field}', class: 'powermail_field_error')}" required="{field.mandatory}" />
with the attribute placeholder causes the exception http://typo3.org/go/exception/v4/1237823695 , what means, that the argument placeholder is not allowed.
Related issues
History
#1 Updated by Helmut Hummel about 2 years ago
Corresponding ticket in TYPO3 CMS tracker #45960
#2 Updated by Alexander Berl about 2 years ago
You should use the additionalAttributes attribute for things like that.
<f:form.textfield ... additionalAttributes="{ placeholder: field.title -> vh:string.RawAndRemoveXss() }" />
#3 Updated by Bastian Waidelich about 2 years ago
Alexander Berl wrote:
You should use the additionalAttributes attribute for things like that.
Right, this is a work around. But I agree that we should make common HTML5 attributes available in the form View Helpers for convenience
#4 Updated by Philipp Maier about 2 years ago
- % Done changed from 0 to 100
This is fixed. Just tested it today with current master.
#5 Updated by Adrian Föder about 2 years ago
right Philipp, it's solved with I24bb73c5de5a494259171ff289d434892f85e16e on Jan 24; however I wonder why this issue is that "fresh", Helmut, is your inquiry based on an older state?
#6 Updated by Adrian Föder about 2 years ago
- Status changed from New to Resolved