[TYPO3-core] FYI24: #16388: [reports] Clean up installed services report

François Suter fsu-lists at cobweb.ch
Sun Nov 14 19:39:43 CET 2010


Hi,

> Problem / Solution:
> The code of the installed services report is a mess, but that's likely
> related to its origins and age.
> Also make sure to adopt styling to other UI elements in the backend.

Thanks for cleaning this up, indeed it didn't look very good. And it's 
true that there was old code in there.

A couple of remarks:

- maybe you could just drop the "ç" from my name and use a straight "c". 
I nearly never use it inside code files, but sometimes the habit gets 
the best of me ;-) I think not using it saves some potential encoding 
problems, even though we now use UTF-8 in the core.

- I'm not convinced about having the availability class apply to the 
whole table row. It used to be applied only to the last cell. I think 
applying it to the whole row makes it too striking visually. What do you 
think?

Cheers

-- 

Francois Suter
Cobweb Development Sarl - http://www.cobweb.ch


More information about the TYPO3-team-core mailing list