[TYPO3-core] RFC: #609 Icon inconsistencies (Shortcut, Undo, Reload)

[Jens Hoffmann]

...

After a discussion with Stucki:

1. This change will NOT go in to 4.2.x
2. This change will get discussed by the
    official HCI Team Members for 4.3.

So, this patch is for NOW .. obsolete.

#609 is the Right Bug ID but was submitted
thought the wrong Tracking System. My fault.
http://forge.typo3.org/issues/show/609

All Icons should follow one standard:
- Same image size (16px x 16px) - be consisted
- Same file type (gif) - 256c are enough.
   1. You don't have enough space to be so
      detailed that you need more colors
      and softer gradients with more than
      256 color. You won't see this.
   2. Well done, sharp Icons dousn't need
      more than 1 alpha state.
   3. PNG24 - JS fix is not needed.
   4. IE6 got still more than 60% so
      we can't skip him ... for now!
      And beleave me, as Design,
      I would love to do so.

Greez Jens

PS: @Ingo, a veto was the wrong tool, from
     my point of view. You could just tell
     me your reason, like Stucki did, and I
     will off take my Patch for good reasons.
     Sorry, but I hope we could discuss such
     disagreements in a more civil way in the
     future.

||||||||||||°||||||||||||

Jens Hoffmann
[TYPO3 - HCI Team Leader]
[DKD - Creative Director]

eMail: jens at typo3.org
Skype: WrYBiT



Jens Hoffmann wrote:
> Hy,
> 
> this is a HCI-related SVN patch request.
> 
> Branches:
> Trunk (4.3) and TYPO3 4.2
> 
> BT reference:
> http://forge.typo3.org/issues/show/609
> 
> Problem:
> Icons are not clear enough.
> 
> Solution:
> See the Icons attached.
> 
> Notes:
> This is one of the bugs I mentioned in my "evil PDF"
> and was fixed during the Dirty Hands On session here
> at NetCreators. Thanks guys!
> 
> 
> ------------------------------------------------------------------------
> 
> Index: /Users/jens/Webserver/typo3core/typo3/classes/class.shortcutmenu.php
> ===================================================================
> --- /Users/jens/Webserver/typo3core/typo3/classes/class.shortcutmenu.php	(revision 3713)
> +++ /Users/jens/Webserver/typo3core/typo3/classes/class.shortcutmenu.php	(working copy)
> @@ -118,7 +118,7 @@
>  
>  		$shortcutMenu = array();
>  
> -		$shortcutMenu[] = '<a href="#" class="toolbar-item"><img'.t3lib_iconWorks::skinImg($this->backPath, 'gfx/toolbar_shortcut.png', 'width="16" height="16"').' title="Shortcuts" alt="" /></a>';
> +		$shortcutMenu[] = '<a href="#" class="toolbar-item"><img'.t3lib_iconWorks::skinImg($this->backPath, 'gfx/toolbar_shortcut.gif', 'width="16" height="16"').' title="Shortcuts" alt="" /></a>';
>  		$shortcutMenu[] = '<div class="toolbar-item-menu" style="display: none;">';
>  		$shortcutMenu[] = $this->renderMenu();
>  		$shortcutMenu[] = '</div>';
> 
> 
> ------------------------------------------------------------------------
> 
> 
> ------------------------------------------------------------------------
> 
> 
> ------------------------------------------------------------------------
> 
> 
> ------------------------------------------------------------------------
>