Opened 8 years ago

Closed 4 years ago

#13591 closed enhancement (patchwelcome)

ResizeHandle constrainAxis should not set width/height for unused axis

Reported by: Nick Fenwick Owned by:
Priority: high Milestone: 1.13
Component: DojoX Layout Version: 1.6.1
Keywords: resizehandle, resizeaxis Cc: liucougar
Blocked By: Blocking:

Description

A div with no width specified resizes itself according to the width of its parent.

If the div is controlled by a ResizeHandle? with resizeAxis: y, the resize method sets both width and height. This fixes a width value, meaning the div no longer resizes with the parent.

Sandbox demonstrating problem: http://dojo-sandbox.net/public/f33ff/0

Any fix would need attention where the resizeHelper.resize() is called, both during activeResize=true dragging, and on the final mouseup resize.

The behaviour can be worked around by setting animateSizing: false so that onResize is the final call, and setting width to 'auto' in onResize. When animateSizing: true (the default) onResize is called before the final width/height is set, so its work is undone.

Sandbox with workaround: http://dojo-sandbox.net/public/f33ff/1

Change History (2)

comment:1 Changed 7 years ago by Douglas Hays

Cc: liucougar added

liucougar, can you own this?

comment:2 Changed 4 years ago by dylan

Milestone: tbd1.12
Resolution: patchwelcome
Status: newclosed

Given that no one has shown interest in creating a patch in the past 4+ years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.