DEfusion.org.uk

tirengarfio,
I managed to use this cropper with the lytebox2. It is allows for additional content rather than just images like in line HTML, new pages and iframes. Use this with your cropper on a different page for a seamless application.

Good Luck

Steven R:

thanks for your recomendation, but could you give a link? Anyway I prefer an open source one.

Javi

Is there a way for me to increase the size of the preview image, rather than have it be minWidth x minHeight?

Javi,

Check out http://www.dolem.com/lytebox/, this just help you do what you are trying to acheive. It worked for me perfectly.

Steven

please disregard my previous, I delved into the code and changed it to suit my needs by changing the Cropper.ImgWithPreview subInitialize and subDrawArea functions.

Hi Dave its a good work.

But when i

give path of correct path of scriptaculous.js by removing the google API’s , it does not work pls help!!

hello dave,

does the cropper ui allow to define a moving area or margins at the top, left, bottom, right, so that the selection can be only moved within this area and not within the whole image?

thx,
ron

Hitesh: There must be a problem with your path or version of script.aculo.us or prototype. Please ensure they’re actually being loaded.

Ron: No the cropper as released does not support this, however if you look at the boundary checking stuff I’m sure you can easily add this functionality to use custom supplied boundaries/margins.

Realy this is great work.

I think I found a bug here.. Or maybe it is a feature.. however, it’s very annoying in my usecase. If anyone knows a way around it without adjusting the cropper.js itself, please tell me.
onEndCrop gets called every time you click somewhere on the page. See for yourself: add

in the callback function in one of the demo’s.. you’ll see an alert wherever you click on the page.
Why is this? isn’t it supposed to be called when you release the crop or a handle?

Any workarounds?
thx in advance.

rv0: This is discussed in the Google Group (http://groups.google.com/group/javascript-image-cropper-ui/browse_thread/thread/bce25c366480b6be) with reasons for it working that way and details of a possible work around.

Please discuss it on the Google Group if you want more details on this.

thanks for pointing me in the right direction.. i’ll get back on that in the google group.

Hello,
is there a way to dynamically change the selected area?

I want to have two radio buttons such as “Use selected area” and “Use whole image”.. and when the user chooses the latter i want to change the dimensions od the cropper to cover the whole image instead of the initial size it loads with.

thanks

John, take a look at the examples which remove and re-apply a cropper you should be able to rework those for your situation.

There doesn’t seem to be a way to destroy the instance that is created. This would be useful for this being built into AJAX applications.

Hi Dave,
thanks for the reply. I’m looking at the demo and trying to merge it with my preview code but having a bit of a trouble..

here is the code i currently have (straight from demo).. now where would i put the function to when the user clicks it, it would resize the cropper to the height and width of the picture?

thank you
[lang]
function onEndCrop( coords, dimensions ) {
$( ‘x1’ ).value = coords.x1;
$( ‘y1’ ).value = coords.y1;
$( ‘x2’ ).value = coords.x2;
$( ‘y2’ ).value = coords.y2;
$( ‘width’ ).value = dimensions.width;
$( ‘height’ ).value = dimensions.height;
}

/**

* Attaches/resets the image cropper
*
* @access private
* @return void
*/

// example with a preview of crop results, must have minimumm dimensions
Event.observe(
window,
‘load’,
function() {
new Cropper.ImgWithPreview(
‘testImage’,
{
minWidth: 200,
minHeight: 120,
ratioDim: { x: 200, y: 120 },
displayOnInit: true,
onEndCrop: onEndCrop,
previewWrap: ‘previewArea’
}
)
}
);
[/lang]

Hi,

i cannot see properly the image cropper in IE6/7/8. In Firefox no problem.

This is my attempt:

http://www.tiregarfio.byethost17.com/web/frontend_dev.php/miembros/cambiarAvatarOK/id/1

Login: fer
Password:m

I have just installed Windows XP/IE6/78 in a virtual machine on Linux,
so any comment or idea or whatever will be wellcome. :)

Regards

Javi

I’m trying to build in a zoom selector which will load a larger version of the image to be cropped if the user wants to. SO far I have been unable to call anything that would re-initilize the cropper and define the greyed out area as part of the bigger image, the selection area remains at the size of the original image loaded… How do I do this?

David: I’m not sure what you’re trying to achieve nor what the problem you are experiencing is. Can you please post your question on the google group (http://groups.google.com/group/javascript-image-cropper-ui/) with either a live example or some code snippets.

Hi,

I wanted to know how to use this utility.

I want to save the resulting cropped image as a new file. Is that possible or is it for display only?

Thanks
Farhan

Hi!
Is someone tested this library on IE7-8, seems it have a bug on it.
On my site cutting area didn’t displayed in IE 8, especially if you call this page by AJAX. for all other browsers it works good (didn’t tested on ie6) :(.

Hi!
I would like to use this script but there is one important problem I didn’t solve. How can I improve quality of cropped image? Default it use approx. 50% compresion and it is too much.
Can you help me?
Thanks

Hi Peter,

The cropper UI doesn’t perform any cropping of the image at all. Please look at your server side implementation to figure out why it is compressing all images at 50%.

First of all. I really love your work. Nice Job!
But I have the following problem.
Before the cropper and all the other UI stuff code i inserted a function for making thumbnails (PHP). The function itself is working but it makes the cropper unfunctional. Everything is displayed as usual but the select frame isnt there. Even if i use an image that wasnt even touched by my PHP function the cropper wont show.

Any idea?

Mike,

Really there is not enough information there that I can even guess at your problem. I suggest checking that the files are included properly and you are not getting and JavaScript errors. If you continue to have problems please post more details on the Google group.

Dave,

I understand what you mean. Can you give me the link to the google group?

Mike,

It is in the links section at the top of the page and in the Next Steps > Feature Requests & Bug Reports section just before the comments.

Dave, thank you very much for an excellent cropping-solution! :)

The crop area changes or resets when the user clicks outside the crop area. Is it possible to disable this behavior so the crop area doesn’t change?

(I haven’t been able to find this question in the comments nor in the Google Groups-discussion.)

Henrik,

That sounds like an integration issue as I’ve not heard any mention of that before. Please post a link, an example or more detail on the Google group.

Thanks for your reply!

The same thing happens on your demo page with Firefox 3.6.3, Opera 10.53 and IE 8.0.

How to recreate (please take a look at the links for screenshots):
1) Define a crop area: http://img17.imageshack.us/i/jscropperui1.jpg/

2) Click somewhere outside the crop area. The crop area is changed: http://img64.imageshack.us/i/jscropperui2.jpg/

Sorry for double posting! :(

I couldn’t see the comment I made yesterday, so that’s why I made another comment. After posting the new comment I could suddenly see it again.

Please delete this comment and the comment I just made.

Henrik,

That functionality is by design, you can click and drag on an area of the image to select an area to crop (or click & drag to move, or on the handles to resize the selected area). In fact that is the entire purpose of the cropper.

Hi,

I think I found a little bug…

When the image is changed dinamically, and the [displayOnInit] is set [true], the new crop definition is not removing the old crop mark.

So far this situation is occuring only with IE8. For Google Chrome it works fine; the old mark desappear and the new mark show up.

If somebody wants to reproduce this bug, please set [displayOnInit = true] and a valid [onloadCoords] on “example-DynamicImage.htm” test page.

Thanks,
Rubens Cury

Just one important consideration…

This bug only will occurs if the [src] image control was not set; In other words, when your [src] image control has an empty image source value.

To solve this bug, instead of leaving an empty image source, set some transparent image and everything works as expected when you change it.

Hope it helps,
Rubens Cury

Dave, thanks for your reply! You are absolutely correct! Don’t ask me what I was thinking about. ;)

hi,

I had a project where I needed in addition to crop, I needed to zoom and scroll around the image, THEN crop a section of it, when the user uploaded a huge image.

in order to scroll around, I had two choices to put scrollbars and to let the crop area itself be moved around and when it touched the sides, to scroll the image.

What I didn’t find in the system, because I don’t think you can normally do this, is the ability to hook on the drag event when you move the crop area around, I found a way in the end, it’s like this

var po = this;
var cd_draw = CropDraggable.prototype.draw;
CropDraggable.prototype.draw = function(point){

po.moveArea(
this.element.positionedOffset(),
this.element.getDimensions()
);

return cd_draw.bind(this)(point);
};

moveArea is just a class method which takes the current position and dimensions of the crop area, so it can calculate where you are in the image for scrolling and thing like that.

I am sure that this is a hack, but if you also need to hook on the drag event for moving the crop area, try this out and I hope it works great for you!!

if there is a better way to do this, I would love to see it. But everything I tried didn’t work, at the end, this worked, so it’s ok with me.

chris

please help i get these errors when i try load the page with the cropper code.

Error: Draggable is not defined
Error: Cropper is undefined.

i’m not sure what the problem is, thanks Dave

Greg:

Please ensure that all the dependencies (Prototype, script.aculo.us, etc.) are included – the error messages you’re getting clearly point to the fact that something in the core of script.aculo.us (Draggable) isn’t there.

For those who have problem with IE8 and not showing cropper (and IE7 worked), make sure you use prototype 1.6.1.0, ruby on rails (ruby framework) uses 1.6.0.x branch by default…

please help me ,I want two or more previewWrap and not the same size, How should I do? Thanks!!

Starho:

I know someone has done this in the past – they did a custom preview class I think. I’m not sure if it’s in the comments or on the google group. But if you take a look at the preview class and how it works you should be able to write your own pretty easily.

Hi & Thanks !...

1. sorry for my Very bad English :$

2. I use cropper on img in div in position “fixed” and your event don’t accept scroll … if you’ve a parameter, thx to inform me, without I inform solution here

++

In reply of post no: 679
Henrik, it is possible to stop creating a new crop box if user clicks outside the current crop box. But if you do that, then resizing of the current crop box doesnt work properly. So resizing has to be stopped too. So, if you want:
-User cannot resize existing cropbox.
-If user clicks outside the current cropbox, nothing happens.
-User can only drag the existing cropbox.

Then change the two methods “startDrag”, and “startResizing” this way:
[lang]
startResize: function(e) {

this.startCoords = this.cloneCoords(this.areaCoords);

//For WTP-1410: If allowzoom is not allowed then user can’t resize cropbox
if(document.getElementById(HiddenZoomTextBox).value==”1”)
{
this.resizing = true;
}
else
{
this.resizing = false;
}
this.resizeHandle = Event.element(e).classNames().toString().replace(/([^N|NE|E|SE|S|SW|W|NW])+/, ‘’);
// dump( ‘this.resizeHandle : ’ + this.resizeHandle + ’\n’ );
Event.stop(e);
},

/**
* Starts the drag
*
* @access private
* @param obj Event
* @return void
*/
startDrag: function(e) {
//mar 1686 allow create new cropbox, and drag that one
if(document.getElementById(HiddenZoomTextBox).value==”1”)
{
this.selArea.show();
this.clickCoords = this.getCurPos(e);

this.setAreaCoords({ x1: this.clickCoords.x, y1: this.clickCoords.y, x2: this.clickCoords.x, y2: this.clickCoords.y }, false, false, null);

this.dragging = true;
this.onDrag(e); // incase the user just clicks once after already making a selection
Event.stop(e);
}
else //mar 1686 dont allow creating a new cropbox
{
this.dragging=false;
}

},
[/lang]

HiddenZoomTextBox is a hidden text box that keeps track of current user permission. It worked for me. Hope it helps.

Hi

I change in Cropper.Img.setParams()
setParams:function(){
var dimensions = this.img.getDimensions();
this.imgW = dimensions.width;
this.imgH = dimensions.height;

...
}

to resolve a Bug in FF 3.6.3 when :

$(‘img’).width => “600px”;
$(‘img’).height => “300px”;

$(‘img’).width => “600px”;
$(‘img’).height => “300px”;

@+

Hi Dave,

Thanks for the great work. The online passport photo DYI tool 123PassportPhoto.com is using your cropper script. It has helped tens of thousands of users from many contries such as UK, US, China, Germany, etc. making their passport photos. It helps to save 1M+ for the users. Without your script, this success cannot be possible. I have added a link to your site in the about page.

Thanks & regards,
Alan

Hi Dave,

In the previous verion, there is a bug in which the cropper sometimes does not show the picture. User need to refresh the webpage. You may want to try it at http://www.123PassportPhoto.com. I will upgrade the cropper to the latest version to see whether it helps to resolve the problem. BTW, if you plan to add a page listing websites that use your script, please add 123PassportPhoto.com.

Thanks & regards,
Alan

its giving me this error i have include all the mentioned thing within my code
Draggable is not defined
[Break on this error] var CropDraggable = Class.create(Draggable, {
cropper.js (line 134)
Cropper is undefined
[Break on this error] new Cropper.Img(