Feature #38752
Task #38755: Usability issues collected on the UX sprint
TCEforms: Replace "field has changed" icon with CSS class
| Status: | Resolved | Start date: | 2012-07-08 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Benjamin Mack | % Done: | 100% |
|
| Category: | - | Spent time: | - | |
| Target version: | - | |||
| PHP Version: | Sprint Focus: | |||
| Complexity: |
Description
When editing a record in TCEforms, each field that is modified gets an arrow symbol in front of the input/select field. There are certain downsides to this:
- Having images and hard-coded srcs in each field clutters the HTML and has ugly JS to change the field, plus it's hard to style
- This does not work for FlexForms
The solution adds a generic TCEforms jQuery object that hooks into the existing fieldChanged function, and does the replacement through CSS classes. The CSS classes now, according to current design standards, just adds a border around the modified image.
Associated revisions
[TASK] FormEngine: "field has changed" icon via CSS
When editing a record in FormEngine, each field that is
modified gets an arrow symbol in front of the
input/select field.
There are certain downsides to this implementation:
- Having images and hard-coded srcs in each field
clutters the HTML and has ugly JS to change the field
- it's hard to style / exchange the icon
The new solution does the replacement via CSS classes.
The CSS class just adds a blue background to the
changed form field or does a color change
to the checkbox label currently.
Releases: master
Resolves: #38752
Change-Id: I94b6cc8550f7539961aa7ee97a8081e7d0ae5931
Reviewed-on: http://review.typo3.org/12708
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
History
#1 Updated by Gerrit Code Review about 3 years ago
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12708
#2 Updated by Gerrit Code Review about 3 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12708
#3 Updated by Gerrit Code Review about 3 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12708
#4 Updated by Steffen Ritter about 3 years ago
- Parent task set to #38755
#5 Updated by Steffen Ritter about 3 years ago
- Assigned To set to Benjamin Mack
#6 Updated by Gerrit Code Review over 1 year 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/12708
#7 Updated by Gerrit Code Review over 1 year 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/12708
#8 Updated by Gerrit Code Review 10 months ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/12708
#9 Updated by Gerrit Code Review 10 months ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/12708
#10 Updated by Gerrit Code Review 10 months ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/12708
#11 Updated by Gerrit Code Review 10 months ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/12708
#12 Updated by Gerrit Code Review 10 months ago
Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/12708
#13 Updated by Gerrit Code Review 9 months ago
Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/12708
#14 Updated by Gerrit Code Review 8 months ago
Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/12708
#15 Updated by Gerrit Code Review 8 months ago
Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/12708
#16 Updated by Benjamin Mack 8 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 30c362b918299de582932f09ee1a87f10b48b76d.