Task #69748

Use IconRegistry API to register icons

Added by Mathias Brodala about 4 years ago. Updated about 3 years ago.

Status:
Rejected
Priority:
Should have
Assignee:
-
Category:
Code Cleanup
Start date:
2015-09-13
Due date:
% Done:

0%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
easy
Sprint Focus:

Description

Currently icons of all shipped TYPO3 extensions have their icons registered directly within the IconRegistry. This should be refactored to use our own API for registering icons in each extension. (aka "Eating your own dog food".)

This should be done last after all icons have been moved to IconRegistry/IconFactory.


Related issues

Follows TYPO3 Core - Story #68805: Replace current icons in the core by using the new Icon-API Closed 2015-08-06

History

#1 Updated by Riccardo De Contardi almost 4 years ago

  • Target version changed from 7 LTS to Candidate for Major Version

#2 Updated by Frank Naegler about 3 years ago

  • Status changed from New to Rejected

As we start to implement / rollout the IconRegistry, we add all core icons directly to the Registry, to prevent hundred of method calls.

The huge amount of icons could (and will) be a performance issue. The API was designed to register icons from external code.
I will reject this issue for the time being. Please feel free to ping me on slack if you really think this decision was wrong.

Also available in: Atom PDF