Project

General

Profile

Actions

Task #69748

closed

Use IconRegistry API to register icons

Added by Mathias Brodala about 9 years ago. Updated about 8 years ago.

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

0%

Estimated time:
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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Riccardo De Contardi almost 9 years ago

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

Updated by Frank Nägler about 8 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.

Actions

Also available in: Atom PDF