[TYPO3-core] RFC #13379: Sprite Icon API

[Thomas Allmer]

On 02.05.2010 20:59, Benjamin Mack wrote:
> finally committed v10 of the patch to trunk (rev. 7511). Thanks!

wow, can't believe it :)
nice :)

attached is a patch to improve it a little more. Do you need a separate 
Bug Tracker Entry for something like this?

- changed the condition in "mapRecordTypeToSpriteIconClass" so that it 
also works if there is a "typeicon_column" but no "typeicon_classes". 
Like for example tt_news 3.0.1 didn't work. (tt_address always worked)

- removed the second css class name in "getSpriteIconClasses" as it's 
just not needed

- added "typeicon_classes" for various types; be_user, be_groups...


Note:
- there seem to be some icons which have a "wrong" name for example 
"status-user-admin" should probably be "mimetypes-x-user-admin"... where 
to discuss such things?

- this icon "apps/pagetree-page-mauntpoint.png" could be removed?

> Next steps:
>
> * Create a sprite generator
> * Make the core icons backwards-compatible so that the core can live
> with calls to the sprite API when there is no skin installed
> * Replace all the parts in the core to make use of the new Sprite API.
>
> If anyone is up for any of the tasks, let me know.
>
> I will now update the documentation on the wiki!
>
> All the best,
> Benni.


-- 
+---------------------------------+-----------------------------------+
| Thomas Allmer                   |   http://www.delusionworld.com    |
| E-mail: at at delusionworld.com    |   phone: +43 699 16217064         |
+---------------------------------+-----------------------------------+
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: spriteIconAPI_improve.patch
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20100503/0628a43c/attachment.asc>