[TYPO3-core] RFC: #15464 no-brainer: SysFolder and NotInMenu-Pages not visible in Pagetree

[Jeff Segars]

On 9/20/10 6:32 AM, Dmitry Dulepov wrote:
> Hi!
>
> Sven Juergens wrote:
>> See attached images (ie6 don't respect the height:16px so before and
>> after looks the same)
>
> I do not understand why having the height causes this effect. Any ideas? It
> is not good to remove it without understanding why the problem happens. It
> is curing symptoms, not the curing the problem.
>

The fundamental problem here is that the additional content for "Not 
shown in menu:" is added inside the first treeLinkItem and is tied 
directly to the first not in menu page.  Typically, a treeLinkItem is 
just the page icon and text (a single row) so the 16px height is OK. 
When we end up with 2 rows inside the treeLinkItem (not in menu text + 
page icon and text) then the height is a problem.

To get to the root cause, we'd need to fix the tree rendering so that 
the "Not shown in menu:" message is its own distinct tree item, not one 
that piggybacks on other elements.

Otherwise, we either need remove the height as Sven suggested or specify 
a height when the not in menu text is present.

The attached v2 patch specifies the height. If there's concern about 
breaking things by removing the height, this should help a little 
without getting into the guts of the tree rendering.

Thanks,
Jeff

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 15464_v2.diff
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20100921/59e19e48/attachment.asc>