Opened 11 years ago

Closed 11 years ago

#5763 closed defect (duplicate)

test_FloatingPane.html and Resizing

Reported by: guest Owned by: dante
Priority: high Milestone: 1.1
Component: Dojox Version: 1.0
Keywords: FloatingPane resizing Cc:
Blocked By: Blocking:

Description

I just tested the "test_FloatingPane.html" (dojo/dojox/layout/tests/test_FloatingPane.html in Dojo 1.0.2 release) in MSIE 6.0 on WinXP and the attempt to resize any resizable pane caused a runtime error "Invalid argument". The same test HTML page worked fine in FF 2.0.0.11.

Change History (5)

comment:1 Changed 11 years ago by Adam Peller

Owner: changed from Adam Peller to dante

comment:2 Changed 11 years ago by mrblakwell

i came across this myself, and looked into it, it appears that resize is expecting top/left in addition to width/height, and doesn't account for the possibility of not passing that in.

i looked at contentPane's resize method and it just calls dojo.marginBox(this.domNode,dim) which i tried in the floating pane and seems to work.

haven't figured out how to produce diffs yet (too busy converting from 0.4 to 1.0) so hopefully this comment is helpful enough...

comment:3 Changed 11 years ago by mrblakwell

oops. looks like you also need to:

dojo.marginBox(this.canvas,{w:dim.w,h:dim.h - this.focusNode.offsetHeight})

so that scrolling works. in my testing before i didn't have very much content.

comment:4 Changed 11 years ago by dylan

Milestone: 1.1

comment:5 Changed 11 years ago by dante

Resolution: duplicate
Status: newclosed

i pretty certain this has been fixed in trunk for a while now, please recheck against 1.1 release which is in b3 now, and will be released very soon.

Note: See TracTickets for help on using tickets.