[TYPO3-core] RFC #10523: New install tool colors, matching t3skin

Susanne Moog typo3 at susannemoog.de
Sun Mar 1 11:49:05 CET 2009


Hi Rob,

Rob Vonk schrieb:
> Patch works with -p0. Tested on 5108.
> 
> I see a tiny markup error on top in 'embedded mode' (see screenshot). In
> FF 3.01 on ubuntu. and IE6 on XP (don't have 7). Full install tool mode
> works because the rest of the background is also white.

Ah thanks for spotting that, I'll fix it in a new version (waiting for
other remarks a bit first).

> 
> I see that you changed fixed markup code into css and imho, thats a good
> thing. Is there a reason that you didn't update all table tags (probably
> time). I see some border="0", align="center".

I just cleaned the parts up I actually worked with and needed for the
new version and left the others as is. Looking at the class it is in
fact in dire need for a big cleanup patch, but that's probably not
needed as there was some talk about a whole new install tool (what's the
status there?).

Regards and thanks for testing,

Susanne


More information about the TYPO3-team-core mailing list