Doesn't work with Internet Explorer 7 (and/or other browsers, sometimes)

Thank you very much for the testing.

Can you verify you experience the same results on the demo page (http://yhager.com/demos/crop/node/1)? Your results simply put mean the module mostly doesn't work as expected.

The amount of happy users make me think this might be something specific in your environment, but I do not know enough to be sure of that at this time.

I've done some more testing using http://browsershots.org, http://ipinfo.info/netrenderer/index.php, and http://www.browsrcamp.com/ on the demo page, all seems pretty good.

I'll need more info in order to recreate this.

The demo is using the following:
Drupal 6.9
ImageAPI 6.x-1.2
FileField-6.x-3.0-alpha6
Imagefield-6.x-3.0-alpha4
Imagefield_crop-6.x-1.0-beta2

yeah, I know.. I should update that site.. oh.. well.. will add that to my ever expanding list of tasks.. :)

Could be. I'll update the demo and check. Thanks again for your report.

Actually it doesn't seem to work quite right in Firefox either. It works great as long as the "The resolution to crop the image onto" is kept fairly narrow. However once the width resolution gets up to about 390 the Crop Area begins to overlap the preview. If the Crop Area is larger than the preview, it can potentially completely cover it. At least that has been my experience.

Firefox 3.0.10.0

Drupal 6.12
ImageAPI 6.x-1.6
FileField 6.x-3.0
ImageField 6.x-3.0
Image crop 6.x-1.x-dev AND Image crop 6.x-1.0-beta3

EDIT:
Created New Issue

@jared12: Please open another issue.

@Onopoc: I have upgraded the demo, and tested
* Safari 3.12 (using http://www.browsrcamp.com/)
* IE6 (on Linux)
* Opera 9.64 on Linux
* Firefox 3.0.8 on Linux
* Konqueror 3.5.9.
* Google Chrome (on Linux, using crossover)

Also, using http://browsershots.org/http://crop.yhager.com/node/1/edit, I can see it looks good on IE6, IE7 and others too.

Can you retest the demo page and let me know your results?

@yhager, for your request:

Tested "Edit" in http://crop.yhager.com/node/1

Works perfect on all browsers i tested EXCEPT for IE8.

Passed test on:
Safari 4.0 Mac
Firefox 3.0.10 Mac, Linux
IE6 XP
IE7 XP

In IE8 the crop area cannot be moved or resized.

Can't test "Add" on your demo site of course.

@tombigel: You can now login on the demo page on '/user' with username 'add' and password 'demo'. Then you can go to 'node/add/page' and add a page. I'll appreciate any help on where is the problem or any hints how should I go about fixing it. Thanks.

@onopoc: Thanks a lot for the detailed bug report. You are setting a very high standard in bug reporting and detecting, and you leave the bug no choice but to be fixed. Thanks!

@Onopoc: Would you be so kind to retest the demo page? Note you can also add images now (the username and password is in comment #15).
I've upgraded Jcrop to latest (0.9.8), which should fix a few issues. I managed to test this using IE7 on crossover (thanks to the new beta they released today!), and it looks like working. If this works for you too, I'll upgrade the Jcrop version for the module and commit.

@Onopoc: Important: make sure you clear your browser cache before you start, otherwise it might not pick up the updated code.

IE8 Compatibility view, Firefox 3.0.11 and Opera 9.64 are doing great.
@Chrome doesnt show the crop image
@IE8 normal view does show the crop image and the crop border, but you're not able to crop it.

I am actually experiencing the same thing with the current versions of all needed modules installed on D6-13

I found a good way which is easiest to solve the problem, it works for me to the day.
just reset the timeout on line "setTimeout(attachJcrop, 700, context);" in file imagefield_crop.js. you can change it to 5000 as your test to make the crop work correctly, hope this will help you

I think we need to thanks Houwenju, a chinese guy, who found the way to resove it. a monument should be erected in his honor. :)

I got the following when applying the patch

patching file imagefield_crop.js
Hunk #1 FAILED at 2.
1 out of 1 hunk FAILED -- saving rejects to file imagefield_crop.js.rej

I prefer not to increase the timeout to 5 seconds. It means that you will have to wait 5 seconds before you can start cropping the image - it might be too much for some people.

Clearly the method of waiting on a timeout till the image is shown is flawed, however, I was unable to find a better way to handle this.

If you have this problem - go ahead and apply the patch, but I prefer to commit a better solution to this, that does not depend on the file size and user's bandwidth.

I am accepting patches happily :)

subscribe

@yhager:
I didn't follow this one for a long while, so if I'm just blubbering - tell me:

I understand that you have a problem with functions running before the image fully loads?
Why not use the $().ready() function of jQuery to fire a trigger to start the crop functions instead of using a timer? If I remember correctly every element has an onload (onready) function.

I have two sites both on the same version of drupal and Imagefield_Crop. In IE 7 one works perfectly the other I experience the above problem... the tiny graphic on node create but it works on node edit. I can't seem to get the working site to break or the broken site work. I have compared and tried to replicate but I can't seem to figure it out. Any ideas on narrowing down the thing that makes the difference? I have not used any patches on either site.

I've got the latest versions of all the relevant modules and core code as of this date, and my IE8 result is the same as comment #19 - crop edges display but you can't resize. I'm on XP. Let me know if you need more info, or if you have a patch.

I'm experiencing this as well in IE 7.

Try this:

  var cropbox = $('#cropbox', context);
  if (cropbox.attr('complete')) {
    attachJcrop(context);
  }
  else {
    cropbox.load(function() {
      attachJcrop(context);
    });
  }

instead of

setTimeout(attachJcrop, 1000, context);

I did a manual replacement of the code in imagefield_crop.js line 6. It did not fix the problem. I will describe what I'm seeing and attach a screenshot: the crop area boundary ("marching ants") is visible on the left, right, and bottom, but not on the top. The cursor does not change when any part of the image is moused over, which is contrary to expected behavior.

Module: 6.x-1.0-beta3
Browser: IE8
System: WinXP

Hi there,

tested #38/39 with Ie 8.06 and latest Firefox.

Firefox works perfect - IE crashes to the point where I need the task manager to close it.

What a pity, such a great module otherwise.

Best regards,
Robert

subscribing

Hello - has there been any further development in fixing this issue?
Very awesome module and I hope this will be fixed soon.

So how is this going? is it fixed?

I tested this module on my site.
It only works normal IF the field is on the first form of a edit page, if its in another field group, its not working.

Also this module is causing my Wysiwig Editor to disappear (the buttons are there but its invisible) on IE 8.
its rather unusable...

Edit:
about the Wysiwig editor, ti was about the CSS limit in IE
"I searched through, and found that IE has CSS limit of 31 files.
It was hitting that limit, go to Site configuration > Perfomance and enable CSS Optimization
it combines into 1 CSS file (i think), and thus TinyMCE works as well as this image.

When you modify the page theme, turn it off and use firefox for the editing."

The problem with this module still persists tho.
I'm using Chrome, dev version of this module with Drupal 6.
On Chrome or on IE, either way you MUST put the imagefield on the first vertical tab of the editing window, so that its always open at the start of the page view, or it will fail.
As long as its on top, it works.

is this still an issue? is the patch at #38/#39 need for this module to work on IE7?

I am also interested in testing the patch at #38 - I don't have IE to test with - so I will be happy for community testing here.

@Hiroaki - Please open a different issue for the issue you described in #45

@dalehgeist - Can you check if you get javascript errors?

i am not experiencing any issues on IE7 and IE7 with latest dev of this module
thanks yhager!

@robby.smith, if you have no issues, let's leave this at "needs more info" for the others, who do.

o sorry about that! i hope people with issues will be able to provide info, then i will help replicate with settings and troubleshoot
thanks!

I was having trouble with the tool in Firefox 3.6 on Ubuntu 9.10. The patch from #28 helped me, but as folks say it is not a perfect solution.

Is there any way to see if the fadein is done? If so we could set the timeout back to 1000 and loop around it until the fadein is finished.

as per #49