Opened 13 years ago

Closed 12 years ago

Last modified 10 years ago

#1953 closed defect (fixed)

Tree inside Accordion causes odd layout problems

Reported by: joose Owned by: Adam Peller
Priority: high Milestone: 0.9
Component: Dijit Version: 0.4
Keywords: AccordionContainer, Tree Cc:
Blocked By: Blocking:

Description

Hi,

I don't know what is the problem with AccordionContainer? and Tree, but when adding new nodes to Tree, it does not update the "size" of the tree box. I took a screenshot of it:

http://joose.iki.fi/Screenshot_10.png

This is only problem with FF, not with Opera/Safari?/IE.

If I call accordion.onResized(); it does not fix it. BUT, I push the button or move SplitPane? or click on menu item.. it fixes it. Odd problem. Tried to track it down, but I was not able.

Using latest trunk. Tree is not TreeV3 tree.

Change History (7)

comment:1 Changed 13 years ago by joose

And, when removing nodes, it does not "resize" back to smaller, until user does something, like moves SplitContainer?

comment:2 Changed 13 years ago by bill

As usual, we need a testcase.

comment:3 Changed 13 years ago by bill

Milestone: 0.4.10.5

Joose is working on a testcase but he hasn't been able to reproduce the problem in a testcase yet. (The testcase he wrote works correctly.) This seems like a corner-case bug; I'm gonna move it to 0.5

comment:4 Changed 13 years ago by brian duchek

There's a testcase built into the last release that exhibits this problem. Broken: http://download.dojotoolkit.org/release-0.4.1/dojo-0.4.1-kitchen_sink/tests/widget/tree/test_Tree_3.html Works: http://archive.dojotoolkit.org/dojo-2006-12-06/tests/widget/tree/test_Tree_3.html

There's something about how the height of the overall tree is being calculated when the nodes are inserted/shown via the "wipe" method that's preventing it from expanding vertically. If you use a different toggle method, then it's not a problem.

comment:5 Changed 12 years ago by Adam Peller

Component: WidgetsDijit
Owner: changed from bill to koranteng

comment:6 Changed 12 years ago by Adam Peller

Owner: changed from koranteng to Adam Peller

Sorry, this one got stale again. Joose, do you know if this is still an issue? Can you provide a test case?

comment:7 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: newclosed

ok, in theme tester, if you expand the tree nodes fully, it's enough to cause a scrollbar in the accordion pane, and it works ok. Is that what you meant? Both Tree and Accordion have been rewritten in 0.9.

Note: See TracTickets for help on using tickets.