Opened 10 years ago

Closed 10 years ago

Last modified 9 years ago

#8698 closed defect (fixed)

dojox.layout.GridContainer does not call startup() on child widgets

Reported by: Shane O'Sullivan Owned by: dante
Priority: high Milestone: 1.3
Component: DojoX Layout Version: 1.3.0b1
Keywords: Cc:
Blocked By: Blocking:

Description

When GridContainer?'s startup() function is called, it should call startup() on all child widgets. Since it does not, any layout widgets inside it that rely on startup being invoked do not work.

A patch is attached that fixes this issue

Attachments (1)

GridContainer_startup.patch (1.3 KB) - added by Shane O'Sullivan 10 years ago.
Patch to fix this issue

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by Shane O'Sullivan

Attachment: GridContainer_startup.patch added

Patch to fix this issue

comment:1 Changed 10 years ago by bill

I wonder if GridContainer would be better to extend dijit.layout._LayoutWidget.

It seems like it should, from a quick glance at the code and the test file, but I'm not sure.

comment:2 Changed 10 years ago by Adam Peller

Owner: changed from Adam Peller to dante

log should come out of patch

comment:3 Changed 10 years ago by dante

Resolution: fixed
Status: newclosed

(In [16773]) series of patches from Shane O'Sullivan - fixes #8698 fixes #8698 and fixes #8708 - thanks shane! minor cleanups to patches in trac, plus worked with shane to optimize potteddnd portion. !strict

comment:4 Changed 9 years ago by bill

Component: DojoxDojoX Layout
Note: See TracTickets for help on using tickets.