#4199 closed bug (fixed)

Opened February 20, 2009 05:16PM UTC

Closed March 17, 2009 11:22AM UTC

Last modified March 19, 2009 03:30AM UTC

resizable with containment boundary - aspectRatio breaks.

Reported by: tcarnell Owned by:
Priority: critical Milestone: 1.7.1
Component: ui.resizable Version: 1.6rc6
Keywords: resizable containment aspectRatio Cc:
Blocked by: Blocking:
Description

I have a resizable div with the following options set:

  • handles:'n,ne,nw,s,se,sw,e,w'
  • autoHide:true,
  • aspectRatio:widgetState.maintainAspectRatio,
  • grid:[10px,10px],
  • containment:"#container",
  • start: function(e, ui) {...},
  • stop: function(e, ui) {...}

However as I enlarge the div in both x & y axis, if one of the axis reaches the boundary of the container the div should stop expanding (to maintain aspect ratio), but the div continues to expand along the other axes, thus breaking the aspect ratio.

Attachments (0)
Change History (8)

Changed February 21, 2009 03:32PM UTC by scottgonzalez comment:1

milestone: TBD → 1.7
priority: major → critical

Changed February 24, 2009 10:40AM UTC by tcarnell comment:2

sorry, to make it clearer:

  • aspectRatio:true

Changed March 02, 2009 09:07PM UTC by raziel057 comment:3

Could you look at JCrop source -> http://deepliquid.com/projects/Jcrop/demos.php?demo=thumbnail ?

Resizing with ratio seems to be very efficient.

Changed March 08, 2009 02:34PM UTC by rdworth comment:4

milestone: 1.7 → 1.8

Changed March 17, 2009 10:57AM UTC by rdworth comment:5

Added visual test http://jquery-ui.googlecode.com/svn/trunk/tests/visual/resizable/resizable_ticket_4199.html

Changed March 17, 2009 11:21AM UTC by rdworth comment:6

Duplicate ticket #4349 contains patch.

Changed March 17, 2009 11:22AM UTC by rdworth comment:7

resolution: → fixed
status: new → closed

Fixed in r2299

Changed March 19, 2009 03:30AM UTC by rdworth comment:8

milestone: 1.8 → 1.7.1

Merged to 1.7.1 in r2324 and r2325