150876 - Trees do not render contents inside GroupBox

Status: RESOLVED INVALID

(Core :: XUL, defect)

Description

[Rob Sanderson]

When putting a <tree> element inside a <groupbox> it fails to render the
contents of the tree.
The XUL at the URL is from the XUL Tutorial and when not inside a groupbox
renders correctly.
Tested under Mozilla 1.0, Linux.

Comment 1

[Rob Sanderson]

Duh. Sorry. Moving.

Comment 2

[Christopher Hoess (gone)]

reassign

Comment 3

[Brendan Eich [:brendan]]

C'mon, hyatt's on sabbatical and anyway, varga has been a great tree (outliner)
owner for a long time.

/be

Comment 4

[arman]

This one is quite similar to bug 148417.  Perhaps they should be merged.

Comment 5

[Jan Varga [:janv]]

A testcase would be helpful

Comment 6

[Rob Sanderson]

Varga:
> A testcase would be helpful

That's what the URL is :)

Here's the contents of the xul file at the URL:

<window
  id="example-window" title="Example 6.3.1"
  xmlns:html="http://www.w3.org/1999/xhtml"
  xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">


<groupbox>

  <tree flex="1">
  <treecols>
    <treecol id="sender" label="Sender" flex="1"/>
    <treecol id="subject" label="Subject" flex="2"/>
  </treecols>

  <treechildren>
    <treeitem>
      <treerow>
        <treecell label="[e-mail]"/>
        <treecell label="Top secret plans"/>
      </treerow>
    </treeitem>
    <treeitem>
      <treerow>
        <treecell label="[e-mail]"/>
        <treecell label="Let's do lunch"/>
      </treerow>
    </treeitem>
  </treechildren>
</tree>

</groupbox>

</window>


HTH

Rob

Comment 7

[Rob Sanderson]

Arman:
> This one is quite similar to bug 148417.  Perhaps they should be merged.

This is to do with rendering, not events.  It is similar in that it has to do
with  Trees, which doesn't seem enough to make the bugs duplicates and merge. 
I've no clue if the events also break, as I can't see the tree to find out ;)

Rob

Comment 8

[Jan Varga [:janv]]

ah, sorry

your <groupbox> needs flex="1"

Comment 9

[Rob Sanderson]

Urgh, yes, flex=1 fixes.  Possibly Evangelism to remove the need for the flex as
the col headers render without it making it look more than a little strange. 
But bug itself is invalid.

Thanks :)

Rob