Opened 10 years ago

Closed 9 years ago

#10450 closed defect (wontfix)

AccordionContainer: unwanted scrollbar appears in child after accordion resize

Reported by: mark007 Owned by: bill
Priority: low Milestone: 1.6
Component: Dijit Version: 1.4.0b
Keywords: Cc: Adam Peller
Blocked By: Blocking:

Description

Hi,

In chrome 3 and 4, with dojo 1.4rc1, moving the leftmost "splitter" on the page below, first to the right, then to the left, introduces a horizontal and vertical scroll bar in Google Chrome.

Its like as if dragging the splitter back to the left doesn't resize the internal contents, so chrome then puts in the scroll bars.

Attachments (4)

index2.php (2.0 KB) - added by mark007 10 years ago.
functions2.js (478 bytes) - added by mark007 10 years ago.
style.css (515 bytes) - added by mark007 10 years ago.
simplified.html (1.9 KB) - added by bill 10 years ago.
test case with extraneous ContentPanes? removed; works fine in safari.

Download all attachments as: .zip

Change History (8)

Changed 10 years ago by mark007

Attachment: index2.php added

Changed 10 years ago by mark007

Attachment: functions2.js added

Changed 10 years ago by mark007

Attachment: style.css added

comment:1 Changed 10 years ago by bill

Priority: normallow
Summary: Moving Splitter in ContentPane + Chrome = ScrollbarsAccordionContainer: unwanted scrollbar appears in child after accordion resize

Interesting bug. I think the real issue is that AccordionContainer is setting overflow: auto on it's child widget.

There are a couple issues with your test case also though:

  1. It's using the deprecated AccordionPane rather than ContentPane.
  2. it has a bunch of unneeded AccordionPane/ContentPane widgets. Remember that whenever you have a ContentPane or AccordionPane with a single child you can just get rid of it; it's not doing anything useful.
  3. it's missing height and width settings for <html> and <body>

Changing those things makes your testcase work. I'll attach a version of your testcase with those changes that works.

Changed 10 years ago by bill

Attachment: simplified.html added

test case with extraneous ContentPanes? removed; works fine in safari.

comment:2 Changed 10 years ago by bill

Cc: Adam Peller added
Milestone: tbd1.5
Owner: set to bill

The overflow="auto" setting is from [9232] (ticket #3466), although I can't figure out why it was added.

ContentPane already has a rule setting overflow: auto so I don't see a need for AccordionContainer to set it. AccordionContainer has code that sets overflow to "hidden" during the animation and then sets it back to it's original value after the animation finishes. Seems like that should be enough.

comment:3 Changed 9 years ago by Adam Peller

Milestone: 1.51.6

comment:4 Changed 9 years ago by bill

Resolution: wontfix
Status: newclosed

Working fine in Chrome 5 (against dojo 1.4, 1.5, and trunk).

Note: See TracTickets for help on using tickets.