[TYPO3-dev] comments: show header only if comments allowed

[Steffen Kamper]

Hi,

Steffen Müller schrieb:
> On 12.01.2009 11:00 Steffen Kamper wrote:
>> Hi,
>>
>> Steffen Müller schrieb:
>>> Hi.
>>>
>>>
>>> On 12.01.2009 10:38 Steffen Kamper wrote:
>>>
>>>> this problem exist for many extensions, so why don't make a patch for
>>>> core configure that in TS? Could be something like
>>>> config.removePibaseWrap = tx_comments_pi1,tx_mmforum_pi5
>>>>
>>> kickstarter produces this boilerplate code. We should also change it
>>> there for future extensions.
>>>
>> as Dmitry said, it's used for CSS, so removing it in general won't be a
>> good idea, but remove it in core when content is empty is the best
>> solution as it doesn't break any CSS.
>>
> 
> I agree with you, but...
> the class wraps can be annoying even if there *is* some content.
> see my other posting in this thread. I pled for making this configurable
> in general.
> 

I see an additional problem:
pi_wrapInBaseClass wraps content with div _and_ wraps it with comment if 
it's not supressed by TS.
This comment is useful to identify plugin in HTML-source.

I think extension-authors should give this option, it's the same to make 
content of plugin available with stdWrap, this is also a task of 
extension author.

I will provide a simple patch that use the div only when content is there.

vg Steffen