Bug #69398: Forms: Form spinner elements need own icon - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #69398

closed

Forms: Form spinner elements need own icon

Added by Björn Jacob over 8 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Björn Jacob

Category:

Form Framework

Target version:

Start date:

2015-08-28

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

The form wizard has a spinner element, see #69024 for example. Right now we're using the classes ".x-form-field-wrap .x-form-trigger" which has "images/form/trigger.gif" as background image. Introduce a new icon which shows the real functionality of the spinner element.

Files

Download all files

69398_before.png (12 KB) 69398_before.png		Oliver Hader, 2015-08-29 19:26
69398_after.png (13 KB) 69398_after.png		Oliver Hader, 2015-08-29 19:26

Actions

Copy link

Updated by Björn Jacob over 8 years ago

Status changed from New to In Progress

Actions

Copy link

Updated by Gerrit Code Review over 8 years ago

Status changed from In Progress 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 http://review.typo3.org/42912

Actions

Copy link

Updated by Gerrit Code Review over 8 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42912

Actions

Copy link

Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42912

Actions

Copy link

Updated by Björn Jacob over 8 years ago

Tracker changed from Bug to Task

Changing to [TASK] by request of Morton Jonuschat

Introducing a new Icon is probably a [TASK] and thus would not be backported to 6.2.

Actions

Copy link

Updated by Gerrit Code Review over 8 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42912

Actions

Copy link

Updated by Gerrit Code Review over 8 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42912

Actions

Copy link

Updated by Gerrit Code Review over 8 years 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/42912

Actions

Copy link

Updated by Gerrit Code Review over 8 years 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/42912

Actions

Copy link Download all files

#10

Updated by Oliver Hader over 8 years ago

File 69398_before.png 69398_before.png added
File 69398_after.png 69398_after.png added
File 69398_before.png 69398_before.png added
File 69398_after.png 69398_after.png added

Compare the both screenshots which show the situation before and after the change.
That's the reason for me considering this rather as bugfix instead of naming it a task.

Actions

Copy link

#11

Updated by Oliver Hader over 8 years ago

Tracker changed from Task to Bug
Is Regression set to No

Actions

Copy link

#12

Updated by Gerrit Code Review over 8 years ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42925

Actions

Copy link

#13