Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#4334 closed bug (wontfix)

Real World Image Cropper demo is broken

Reported by: nicklevett Owned by: Scott González
Priority: major Milestone: 1.6
Component: ui.resizable Version: 1.6rc2.6
Keywords: image cropper real world resizable Cc:
Blocked by: Blocking:

Description

This demo seems to be broken in both Firefox and IE7.

Firefox shows a 100x100 crop of the picture and you can't resize it. IE7 displays the entire image and you can resize to a certain extent but it's buggy.

In IE7 try dragging the bottom right handle diagonally down - it will stop resizing at ~295px (width). Move the entire selection box to the right hand side of the image and try dragging the bottom left handle - the selection will flip and cause an error (line 130).

Change History (11)

comment:1 Changed 8 years ago by nicklevett

Back in UI 1.5 the width and height calculations are correct and stop at the width and height of the container however 1.5.1 onwards don't do this and goes to whatever size you have dragged. Can this be fixed as well.

Firefox 2 does what IE7 does. Firefox 3 just refuses to work.

comment:2 Changed 8 years ago by Scott González

Resolution: invalid
Status: newclosed

This demo doesn't exist anymore.

comment:3 Changed 8 years ago by nicklevett

Ok fine it doesn't exist anymore (in 1.7+) but resizables is still buggy and should be fixed, especially the containment bug.

comment:4 Changed 8 years ago by Scott González

Resolution: invalid
Status: closedreopened

comment:5 Changed 8 years ago by Scott González

Owner: set to scott.gonzalez
Status: reopenedaccepted

comment:6 Changed 8 years ago by Scott González

Resolution: fixed
Status: acceptedclosed

Fixed in r2376.

comment:7 Changed 8 years ago by rdworth

Resolution: fixed
Status: closedreopened

One more fix was applied in r2381 but the issue remains.

comment:8 Changed 8 years ago by rdworth

Version: 1.6rc2.51.6rc2.6

comment:9 Changed 8 years ago by nicklevett

Discussion on Google Groups.

comment:10 Changed 8 years ago by Scott González

Partial fix in r2448.

comment:11 Changed 8 years ago by Scott González

Resolution: wontfix
Status: reopenedclosed

We've fixed the demo as much as we will for the 1.6 release. We're open to accepting a patch, but aren't planning on putting any more effort into this legacy release.

Note: See TracTickets for help on using tickets.