AccordionContainer + bigger than oneline labels = bug when not selected

[joose]

When label is bigger than one line, then not-selected accordionpanes has wrong background. Tested with Firefox and Safari

[joose]

Patch to fix this bug

[joose]

Seems that there still is a bug, for AccordionContainer? countring position of the labels wrongly when they are minimized. Try makeing more than one label have bigger labels.

[joose]

Did little more testing: AccordionPane?.getLabelHeight() returns wrong height when there is image making label bigger.

[joose]

Found the problem: Problem is that labelHeight is calculated when images in the labels are not yet loaded -> wrong height of the label.

Solutions?

1. define height to all images in label
2. make accordion somehow know when images are loaded and THEN calculate height.
3. Other?

[Adam Peller]

Need a test case. The comments seem to imply that an image in the accordion title is required to see this bug? I think adding code to handle image load events seems a bit heavyweight, and I'd be tempted to go with your #1 (or #3 have the user wire up image onload events and trigger a layout, perhaps?)

[Adam Peller]

ok, I tested multi-line labels (the originally reported bug) and it seems to work fine now. If images are a problem, a test case is needed and we might need to document our way out of that one with a workaround, unless a cheap workaround can be found.