[TYPO3-RTE] Indentation using wrapper DIVs

[Joerg Wagner]

Hello Stanislas,

 > dynamiccss is really deprecated and not used anymore in TYPO3 4.2.

oops, I definitely missed this change!

Just some minor comments on this:

1.
EXT:rtehtmlarea/res/contentcss/default.css also contains no predefined 
class for indenting, so it won't work out of the box. Wouldn't it make 
sense to include that?!

2.
EXT:rtehtmlarea/res/contentcss/default.css still contains defaultcss.css 
as filename in the header comments.

3.
It would be great to add a comment line to 
EXT:rtehtmlarea/htmlarea/plugins/DynamicCSS/dynamiccss.css telling that 
this file is not the recommended default CSS file for RTE any more. 
There are probably many people that compare it to their local copy 
somewhere in /fileadmin every now and then (as I do). Having this 
comment line would point them to the new file to use.

Cheers,
Jörg





Stanislas Rolland schrieb:

> Hi Jörg,
>>
>>  > The RFC is now submitted to the Core Team List.
>>  > You may apply the patch, test it, and give your vote in the list.
>>
>> Done.
> Thanks. This is now committed.
>>
>> Stanislas, wouldn't it make sense to also include a corresponding CSS 
>> rule for indenting into
>> /sysext/rtehtmlarea/htmlarea/plugins/DynamicCSS/dynamiccss.css ?
>> So including this CSS file would render all basic RTE functions 
>> operational.
>>
> dynamiccss is really deprecated and not used anymore in TYPO3 4.2. It is 
> there only in case someone would insist to get the exact same behaviour 
> as in TYPO3 4.1.
> 
> The file to import would now be 
> EXT:rtehtmlarea/res/contentcss/default.css. This is the css file that is 
> used by default in the RTE.
> 
>> I also found that dynamiccss.css contains rules for align-left, -right 
>> and -center, but not for align-justify.
>>
> EXT:rtehtmlarea/res/contentcss/default.css does include the rules for 
> align-justify.
> 
> Regards,
> Stanislas
>