Changes between Initial Version and Version 1 of Ticket #16641


Ignore:
Timestamp:
Jan 31, 2013, 2:53:53 AM (7 years ago)
Author:
bill
Comment:

I don't see why it would make a difference which one we measure.

Please attach a test case using the "attach file" button. It should be as small as possible to still reproduce the problem, almost always a single HTML file that we can load in the browser (i.e. not PHP, JSP, etc.)

Then, give exact instructions on how to reproduce the problem using your attached test file.

The test case is necessary to confirm that there's a bug, and to confirm that your change fixes the bug.

Thanks!

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #16641

    • Property Owner changed from bill to idleWombat
    • Property Status changed from new to pending
  • Ticket #16641 – Description

    initial v1  
    1 The layout function of the dijit.layout.AccordionContainer calculates the new verticalSpace for selected accordion widgets.
     1The layout function of the dijit.layout.!AccordionContainer calculates the new verticalSpace for selected accordion widgets.
    22In my opinion the calculation is wrong, because when summing up the total height of all closed accordion widgets, it uses the height of their domNodes instead of the _buttonWidget height.
    33
    4 A spike solution with a patched version of the AccordionContainer can be watched here:
     4A spike solution with a patched version of the !AccordionContainer can be watched here:
    55[http://jsfiddle.net/idleWombat/cjxpu/40/]
    66
    77To use the patched version see the lines:
     8{{{
    89var accordion = new AccordionContainer();
    910// use patched version
    1011//var accordion = new PatchedAccordionContainer();
     12}}}