Hello,

We are using imagecrop with IE7-9 in Compatibily Mode. There are extra commas in the end of arrays of imagecrop.ui.crop.js which needs to be removed, because IE expecting another.

CommentFileSizeAuthor
#2 imagecrop-js_arrays_extra_commas-7148538-2.patch773 byteshe0x410
#1 imagecrop-js_arrays_extra_commas-1936556-1.patch945 byteshe0x410

Comments

he0x410’s picture

he0x410 commented
StatusFileSize
new imagecrop-js_arrays_extra_commas-1936556-1.patch945 bytes

Here is the patch.

he0x410’s picture

he0x410 commented
StatusFileSize
new imagecrop-js_arrays_extra_commas-7148538-2.patch773 bytes

Disregard patch in comment #1.
Correct one here.

he0x410’s picture

he0x410 commented
Version: 7.x-1.0-rc3 » 7.x-1.x-dev

And I selected wrong version. This patch is for 7.x-1.0-rc3+56-dev snapshot.

ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
commented
Status: Needs review » Reviewed & tested by the community

Really simple fix.

ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x-1.x-dev, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.