Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#1238 closed defect (fixed)

[patch] Accordioncontainer has a bug, here is a fix

Reported by: joose@… Owned by: Adam Peller
Priority: high Milestone:
Component: Widgets Version: 0.3
Keywords: AccordionContainer Cc:
Blocked By: Blocking:

Description

If there are no children, then this is not working.

Index: src/widget/AccordionContainer.js
===================================================================
--- src/widget/AccordionContainer.js    (revision 5157)
+++ src/widget/AccordionContainer.js    (working copy)
@@ -46,7 +46,9 @@
                },
 
                onResized: function(){
-                       this.children[0].setSizes();
+                       if (this.children.length > 0) {
+                               this.children[0].setSizes();
+                       }
                }
        }
 );

Change History (4)

comment:1 Changed 13 years ago by dylan

Milestone: 0.4
Summary: Accordioncontainer has a bug, here is a fix[patch] Accordioncontainer has a bug, here is a fix

comment:2 Changed 13 years ago by Adam Peller

Owner: changed from bill to Adam Peller

comment:3 Changed 13 years ago by Adam Peller

Resolution: fixed
Status: newclosed

(In [5684]) Fixes #1238 (thanks, Joose -- CLA on file)

comment:4 Changed 12 years ago by (none)

Milestone: 0.4

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.