Task #82849

Change svg tree json data to contain icon id instead of content.

Added by Tymoteusz Motylewski about 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2017-10-24
Due date:
% Done:

100%

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

Description

Currently svg tree gets the icon content for each node in the JSON data.
Instead just an icon identifier should be passed.
Thanks to that the JSON response will be few times smaller and it will also speed up rendering.

Associated revisions

Revision a9382eae (diff)
Added by Tymoteusz Motylewski about 2 years ago

[TASK] Make SVG category tree use JS Icon API

This change make SVG tree expect icon identifier instead of icon source
in JSON node data.
Tree fetches icon source via JS Icon API only once per icon type.
This makes JSON response few times smaller.

Resolves: #82849
Releases: master
Change-Id: I8954837f68cee3bcc316bc56e30788e0d16c81d6
Reviewed-on: https://review.typo3.org/54468
Tested-by: TYPO3com <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>

History

#1 Updated by Tymoteusz Motylewski about 2 years ago

  • Description updated (diff)

#2 Updated by Gerrit Code Review about 2 years ago

  • Status changed from New to Under Review

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/54468

#3 Updated by Gerrit Code Review about 2 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/54468

#4 Updated by Gerrit Code Review about 2 years ago

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

#5 Updated by Tymoteusz Motylewski about 2 years ago

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

#6 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF