Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#1229 closed defect (fixed)

SplitContainer sizer stuck on iframe

Reported by: ornus Owned by: liucougar
Priority: high Milestone: 0.9
Component: Widgets Version: 0.3
Keywords: splitcontainer Cc:
Blocked By: Blocking:

Description

It seems that SplitContainer? sizer hits the wall with iframes. The only way to resize is to move the mouse around the frame.

Attachments (3)

splitContainer_1.patch (2.4 KB) - added by ornus 13 years ago.
patch fixing the problem based on the solution suggested earlier
splitContainer_1.2.patch (5.0 KB) - added by ornus 13 years ago.
updated patch. added test case.
1229.patch (4.5 KB) - added by liucougar 12 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 13 years ago by guest

A fix was suggested in this thread. I tried it, and it works well for SplitContainers? with two splits on Mozilla, but does not work well on IE.

comment:2 Changed 13 years ago by bill

Milestone: 0.40.5

comment:3 Changed 13 years ago by ornus

thanks. this solution works. to get it to work in IE set z-index to a positive number. IE might have to be in strict mode before it will work, not sure.

Changed 13 years ago by ornus

Attachment: splitContainer_1.patch added

patch fixing the problem based on the solution suggested earlier

comment:4 Changed 13 years ago by bill

Basically dup of #1044 but leaving open since patch attached.

Changed 13 years ago by ornus

Attachment: splitContainer_1.2.patch added

updated patch. added test case.

comment:5 Changed 13 years ago by guest

The patch did not exactly work for me. I have a ContentPane within a vertically aligned SplitContainer, and this ContentPane contained an IFrame. The lid's height in this case was not extending to cover the whole ContentPane (despite the CSS settings), so I had to explicitly set the dimension with:

showCover: function(/*Panel widget*/ panel){ 
    // ..    
    coverDiv.className="dojoSplitContainerCover";
    dojo.html.setMarginBox(coverDiv, dojo.html.getMarginBox(panel.domNode)); 
}

Changed 12 years ago by liucougar

Attachment: 1229.patch added

comment:6 Changed 12 years ago by koranteng

Owner: changed from bill to koranteng

comment:7 Changed 12 years ago by liucougar

Owner: changed from koranteng to liucougar

steal this one

koranteng: if you are not happy with the patch, feel free to reopen it

comment:8 Changed 12 years ago by liucougar

Resolution: fixed
Status: newclosed

(In [7453]) fixes #1229 by introducing a div to cover the entire area occupied by a splitcontainer. the div is under the sizer

comment:9 Changed 12 years ago by liucougar

(In [8523]) refs #1229: added a test case

comment:10 Changed 12 years ago by liucougar

(In [8524]) refs #1229: some text to explain the test case

Note: See TracTickets for help on using tickets.