Opened 12 years ago

Closed 12 years ago

#3585 closed defect (fixed)

test_LayoutCode.html not working

Reported by: bill Owned by: Adam Peller
Priority: high Milestone: 0.9
Component: Dijit Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

Pressing most of the buttons (at least on IE6) causes one error or another. Need to fix this to test new _destroyElement() function (#2931)

Change History (9)

comment:1 Changed 12 years ago by bill

Owner: changed from bill to Adam Peller

OK, I'm gonna checkin the _destroyElement() changes before this gets fixed, so after fixing please make sure that everything works w/out error on IE6. Actually should check if there is still a memory leak on IE6. (Press the buttons repeatedly and watch if the memory usage of IE goes up without limit.)

comment:2 Changed 12 years ago by Adam Peller

(In [9432]) Some improvements to test_LayoutCode.html to fix Tab Non-Layout. Refs #3585

comment:3 Changed 12 years ago by Adam Peller

(In [9437]) Get operations table working. Refs #3585

comment:4 Changed 12 years ago by Adam Peller

Milestone: 0.9beta0.9
Priority: normalhigh

1) the simplelayout results are different from 0.4.x -- top doesn't stretch from left to right. bug or feature? 2) splitpane and tab layout throw an exception, I think because layout is called the wrong time when children are added after construction 3) the current widgets panel gets longer and longer with each iteration. it didn't used to.

comment:5 Changed 12 years ago by Adam Peller

(In [9465]) Fix lazy initialization of children. Refs #3585

comment:6 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: newclosed

(In [9475]) Use id instead of widgetId. Fixes #3585

comment:7 Changed 12 years ago by Adam Peller

bill says the simple layout changes are due to a change in 0.9 regarding the layoutPriority flag. Everything else is fixed, more or less. We'll track any new issues in a separate bug.

comment:8 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: closedreopened

This still isn't working for Bill. Just press "Simple Layout" followed by "SplitContainer?". I get an exception in firebug because Split Container::addChild() is calling layout() before startup(). Must be in IE.

comment:9 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: reopenedclosed

old copy. it works fine, apparently.

Note: See TracTickets for help on using tickets.