Task #77081

Remove category tree options: width, allowRecursiveMode, autoSizeMax

Added by Tymoteusz Motylewski over 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2016-07-13
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

There are several category tree TCA configuration options which can be removed.

  1. allowRecursiveMode setting

it is not working for a while and the documentation vs implementation was off - see https://forge.typo3.org/issues/77074

  1. recursive selection button
    - it was not widely used as nobody expected that the green "refresh" icon is related to recursive selection
    - When implemented 4 years ago, the purpose of this button was to ease handling o "record storage page".
    But now we have "recursive" select box using which user can set recursion level
  1. Width
    Right now it's 280px by default but can be tuned through TCA.

It should be 100% by default to be more accessible for trees with multiple levels and responsive backend

  1. autoSizeMax

when autosizemax was present, it was used as max-height, and 'size' was used as min-height

autosizemax can be dropped and size can be used as max height


Related issues

Related to TYPO3 Core - Bug #77074: formengine tree appearance allowRecursiveMode has no effect Rejected 2016-07-13
Related to TYPO3 Core - Task #77220: Replace category tree toolbar icons with SVG Closed 2016-07-22

Associated revisions

Revision f7643ec6 (diff)
Added by Tymoteusz Motylewski about 3 years ago

[!!!][TASK] Remove selectTree options:width,allowRecursiveMode,autoSizeMax

This patch removes following TCA options related
to 'renderType' => 'selectTree', FormEngine elements:

  • ['config']['autoSizeMax']
    starting from now 'size' will be used as max-height
  • ['config']['treeConfig']['appearance']['width']
    tree will use up the available space instead
  • ['config']['treeConfig']['appearance']['allowRecursiveMode']
    dropped without substitution

Resolves: #77081
Releases: master
Change-Id: Ic70d24ea5576f89812f9996e697fecee8c752e7f
Reviewed-on: https://review.typo3.org/48960
Tested-by: Bamboo TYPO3com <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>

History

#1 Updated by Gerrit Code Review over 3 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 https://review.typo3.org/48960

#2 Updated by Gerrit Code Review over 3 years 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/48960

#3 Updated by Tymoteusz Motylewski over 3 years ago

if you need bunch of test categories, you can use a sql file from issue https://review.typo3.org/#/c/48060/ which contains tree of 300 categories.

#4 Updated by Gerrit Code Review over 3 years 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/48960

#5 Updated by Gerrit Code Review over 3 years 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/48960

#6 Updated by Gerrit Code Review over 3 years 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/48960

#7 Updated by Tymoteusz Motylewski about 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#8 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF