Bug #66659
closedSorting icons are triggered based on language settings
0%
Description
The sorting arrow icons, that can be used to move one step up and down are triggered by the following code:
$singleElementHTML .= $this->tt_content_drawHeader( $row, $this->tt_contentConfig['showInfo'] ? 15 : 5, $disableMoveAndNewButtons, TRUE, !$this->tt_contentConfig['languageMode'] );
public function tt_content_drawHeader($row, $space = 0, $disableMoveAndNewButtons = FALSE, $langMode = FALSE, $dragDropEnabled = FALSE) {
Besides the fact that "languageMode" and "dragDropEnabled" are referring to completely different topics, this leads to two different problems:
1.) There will be sorting arrows in the language view of the page module, but not in the original view (although drag & drop is possible for each of them)
2.) There is no way to activate or deactivate the sorting arrows by a parameter other than the language mode
To make the backend as accessible as possible, while still removing redundant stuff, the sorting arrows should be triggered by a parameter "showSortingArrows", which should be set to false by default. This will enable moving records up and down for people who can not drag and drop (i.e. because they are using the keyboard only or a screenreader, or both), while it will be disabled for the rest of the users, since they can use drag & drop.
So we should keep the fifth parameter of the method tt_content_drawHeader but change the trigger.
Updated by Jo Hasenau over 9 years ago
Ideally the parameter should be part of User TSconfig, since this would give every user the option to enable the icons for accessibility reasons if necessary.
Updated by Gerrit Code Review over 9 years 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 http://review.typo3.org/39083
Updated by Gerrit Code Review over 9 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/39083
Updated by Mathias Schreiber about 9 years ago
- Target version deleted (
next-patchlevel)
Updated by Benni Mack about 7 years ago
- Status changed from Under Review to Rejected
Sorting in PageLayoutView was removed, no activity in the last 2 years, so this topic will be closed.