Opened 13 years ago

Closed 12 years ago

#2643 closed defect (fixed)

AccordionContainer FF render bug

Reported by: guest (peter higgins) Owned by: Adam Peller
Priority: high Milestone: 0.9
Component: Dijit Version: 0.4.2
Keywords: Cc:
Blocked By: Blocking:

Description

this only seemingly affects FF 2.0.0.2 ... tested with IE6, Opera, and FF:

eg: an accordioncontainer with 3 panes, the first two behave as expected. the third pane, when viewed, then pushed out of view by either pane above it, displays the content outside the container [overflow:hidden], below ... again, only in FF, and it fixes itself if you:

a) take a screenshot (go figure) b) scroll or resize the window

screenshots:

(from trunk) / (from testcase) http://inpdx.net/~dante/accordionerror.jpg http://inpdx.net/~dante/acc_error.jpg

can attach a testcase, but first image is taken directly from /nightly/tests ...

Change History (12)

comment:1 Changed 13 years ago by guest

so so sorry - this is a duplicate of this ticket, ultimately:

http://trac.dojotoolkit.org/ticket/1969

comment:2 Changed 12 years ago by bill

Milestone: 0.9beta
Owner: changed from bill to koranteng

I'm not sure this is the same thing as the ticket listed above.

In any case, I'd like Koranteng to try changing the implementation so that the panes don't overlap each other (with position:absolute and different z-indexes), but rather all the closed panes have height=0, and they _don't_ have position: absolute.

Depends somewhat on #2944 (note the required nested div listed in that bug report)

comment:3 Changed 12 years ago by Adam Peller

Component: WidgetsDijit

comment:4 Changed 12 years ago by Adam Peller

Owner: changed from koranteng to Adam Peller
Status: newassigned

comment:5 Changed 12 years ago by Adam Peller

(In [9158]) Reworked AccordionContainer? to subclass PageContainer? and make more use of HTML flow layout. Still needs xp work and initial pane doesn't always stay up. Refs #2926, #2643

comment:6 Changed 12 years ago by Adam Peller

(In [9164]) Update themeTester to use new Accordion schema with AccordionPane?. Refs #2926, #2643

comment:7 Changed 12 years ago by bill

(In [9172]) Add back dojo.require("dijit.util.manager"). This shouldn't have been removed. Refs #2926, #2643

comment:8 Changed 12 years ago by Adam Peller

(In [9181]) Fixup some regressions in Accordion/PageContainer?. Refs #2926, #2643

comment:9 Changed 12 years ago by Adam Peller

(In [9182]) one more bad array reference refs #2946, #2643

comment:10 Changed 12 years ago by Adam Peller

(In [9194]) Fix initial Accordion layout, allow for empty PageContainers?. Refs #2926, #2643

comment:11 Changed 12 years ago by Adam Peller

Milestone: 0.9beta0.9

Accordion was rewritten such that this is now only a problem with scrollbars for certain contained, complex widgets, and only in certain versions of FF. I'll come back to this. One option is to resize on each animation step. Or, we could choose not to support this or wait until a future version of FF.

comment:12 Changed 12 years ago by dante

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.