Opened 12 years ago

Closed 12 years ago

Last modified 10 years ago

#3807 closed defect (fixed)

AccordionContainer fails to startup if there is no child

Reported by: guest Owned by: davidb
Priority: high Milestone: 0.9
Component: Dijit Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

AccordionContainer? fails to startup if there is no child, check the line:

this.getChildren()[0].focusNode.setAttribute("tabIndex","0");

at startup().

It should be possible to create an empty AccordionContainer? and add some AccordionPanes? later, after startup (for example programatically).

Change History (5)

comment:1 Changed 12 years ago by Adam Peller

Milestone: 0.9
Owner: changed from bill to Adam Peller

comment:2 Changed 12 years ago by Adam Peller

Owner: changed from Adam Peller to davidb

David, I think the else clause in AccordionContainer?.startup() should go away entirely. this.selectedChildWidget will default to children[0] in the superclass (StackContainer?) so this code is redundant, I think. The only way the if() condtion will fail is if there are no children, then we should do nothing.

comment:3 Changed 12 years ago by davidb

Status: newassigned

comment:4 Changed 12 years ago by davidb

Peller, I agree. Making the change now.

comment:5 Changed 12 years ago by davidb

Resolution: fixed
Status: assignedclosed

(In [9856]) Fixes #3807: removed redundant setting of tabindex property. (Thanks Peller)

Note: See TracTickets for help on using tickets.