[TYPO3-core] RFC: Bug 6625: Cleaner Backend: alt_doc.php Part 1

[Ingmar Schlecht]

Hi Benni,

great patch!

+1 based on testing (Konqui, FF and IE6) and code-review.

Here are suggestions for possible improvements:
 - The undo icon is sometimes there, sometimes not, depending on whether
   an un-doable change exists. That causes the selector boxes to the
   left of the icons to jump left or right when you switch between
   different docs where one has the icon and the other one doesn't or
   when you save a doc that previously didn't have an un-doable change
   recorded

 - I know the Icons are not final, but anyway: The shortcut icon
   definitely looks too much like undo (or redo, if it existed :))

 - Tab headers in the old screen now got smaller. I prefered them
   bigger.

Those are just minor comments though.

cheers
Ingmar



Benjamin Mack schrieb:
> Hey Steffen,
> 
> correct. New patch attached.
> * The white is a light grey now
> * It is patched against root now
> * minor CSS will be cleaned up once people test it. I hope to get more
> feedback once this is in trunk and the next alpha is out. Our general
> Iconset will be streamlined for 4.2 by Jens.
> 
> greetings,
> benni.
> -SDG-
> 
> www.xnos.de // www.xnos.org
> 
> 
> Steffen Kamper wrote:
>> "Benjamin Mack" <mack at xnos.org> schrieb im Newsbeitrag
>> news:mailman.5375.1194512392.20721.typo3-team-core at lists.netfielders.de...
>>
>>> Hey,
>>>
>>> attached is a new version with a couple of things better streamlined,
>>> after more testing by Steffen Kamper and me.
>>>
>>> greetings,
>>> benni.
>>> -SDG-
>>>
>>> www.xnos.de // www.xnos.org
>>>
>> Hi Benni,
>>
>> the patch isn't against root.
>> The changes are good - now the problem with css has gone and all looks
>> fine.
>> So some css-work is left to let it look better, i think jens will
>> provide new icons, and the white has to be grey ?!?
>> What a fun now editing records!
>>
>> my +1 ;-)
>>
>> vg Steffen
>>
>> _______________________________________________
>> Before posting to this list, please have a look to the posting rules
>> on the following websites:
>>
>> http://typo3.org/teams/core/core-mailinglist-rules/
>> http://typo3.org/development/bug-fixing/diff-and-patch/
>> _______________________________________________
>> TYPO3-team-core mailing list
>> TYPO3-team-core at lists.netfielders.de
>> http://lists.netfielders.de/cgi-bin/mailman/listinfo/typo3-team-core