Bug #53594
No placeholder attribute in backend forms
| Status: | Resolved | Start date: | 2013-11-13 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | - | % Done: | 100% |
|
| Category: | Backend User Interface | Spent time: | - | |
| Target version: | next-patchlevel | |||
| TYPO3 Version: | 6.0 | Is Regression: | No | |
| PHP Version: | 5.3 | Sprint Focus: | ||
| Complexity: |
Description
The disposal of the placeholder feature in backend forms needs to be enabled using the mode setting "useOrOverridePlaceholder" in TCA. However, the implementation for checking this needs to be inverted, since it returns an empty string if it's enabled, but should be the opposite way instead.
How to reproduce:- enter a title for a file in the file backend module
- create a new content element (text w/image) and assign that file
- after enabling the override mode, the input field should show the placeholder (using the HTML5 placeholder attribute)
How it should look like (but does not currently):
Related issues
Associated revisions
[BUGFIX] No placeholder attribute in backend forms
The disposal of the placeholder feature in backend forms needs
to be enabled using the mode setting "useOrOverridePlaceholder"
in TCA. However, the implementation for checking this needs to
be inverted, since it returns an empty string if it's enabled,
but should be the opposite way instead.
Fixes: #53594
Releases: 6.2, 6.1, 6.0
Change-Id: I3d3fe5ac76c05f4601860c79d7f514bffea13963
Reviewed-on: https://review.typo3.org/25365
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
History
#1 Updated by Oliver Hader over 1 year ago
- File 53594.png added
#2 Updated by Gerrit Code Review over 1 year ago
- Status changed from New 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/25365
#3 Updated by Gerrit Code Review over 1 year ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25365
#4 Updated by Gerrit Code Review over 1 year 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/25365
#5 Updated by Oliver Hader over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset c631a58fabea7e1ebee5038a4308b47e9b8dc111.