Hello,

I was wondering if this is by design or an issue that is currently being addressed/or needs to be addressed if possible.
The FULL image shows for 1 second after page load, then adjusts to the smaller as per settings (as seen in attachment) to be used to select the image cropping area.

Is there a way to make the full image not show on page load?

Are others also seeing this issue?
I see the same issue using IE, Safari, Firefox

Thank you!

CommentFileSizeAuthor
Untitled.jpg67.01 KBBilmar

Comments

rburgundy’s picture

rburgundy commented

subscribing.

Bilmar’s picture

Bilmar commented

I have tested on a fresh Drupal install.
Still encountering the issue of the cropping preview image on the edit page showing in Full Size before it is re-sized.

Can anyone comment on if they are also seeing this?

guypaddock’s picture

guypaddock commented

I see this as well. I tried setting the following in the style.css, and while it fixed this issue, it broke the cropping:

div.node-form img.cropbox {
  display: none;
}
guypaddock’s picture

guypaddock commented

This seems to fix it:

/* ImageField Crop */
div.node-form img.cropbox {
  display: none;
}

div.node-form div.jcrop-holder img.cropbox {
  display: block;
}
cels’s picture

cels commented
Component: Code » User interface
Status: Active » Needs review

#4 your fix it's OK for me.

I added to imagefield_crop.css and seems to be good.

Bilmar’s picture

Bilmar commented

thanks GuyPaddock! worked like a charm =)

Would anyone be able to comment if this css is quick fix for an issue with the module, or the issue always existed by design from the start?

yhager’s picture

yhager commented

Thanks for the patch!

The reason this happens is that the crop interface can only work after the image has completed download and display - so the module waits about a second for this. IE7 being the most problematic browser in this regard.

However your fix seem to hide it nicely - can anyone confirm it works for IE7+ ?

Bilmar’s picture

Bilmar commented
Status: Needs review » Reviewed & tested by the community

Hello yhager,

Cool - I understand now. Thanks for the explanation =)
Yep, I just tested on IE 7.0.5730.13 and looks good!

Changing status to RTBC for your final review and comments.

Thanks

yhager’s picture

yhager commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

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

Bilmar’s picture

Bilmar commented
Status: Closed (fixed) » Needs work

Installed the newest dev and experienced the issue again.

The change to the css file modified the class node form, but it needed to modify the id node form.
When I changed it to id node form the problem was resolved.

yhager’s picture

yhager commented

@trupal218: Can you make a patch?

Bilmar’s picture

Bilmar commented

As I learn how to create patches, please refer to the below code used to fix the issue.

#node-form img.cropbox {
  display: none;
}

#node-form .jcrop-holder img.cropbox {
  display: block;
}
yhager’s picture

yhager commented

@trupal218: I don't see the value in the change you propose.
The page is structured as follows:

<form id="node-form">
  <div class="node-form">
    <!-- all the rest here -->
  </div>
</form>

so CSS rules like the one existing, or the one you propose should have the same effect anyway. Maybe you have a special theme or theme function which behaves differently?

[edit: I got the div and form backwards, but the argument still holds]

Bilmar’s picture

Bilmar commented
Status: Needs work » Fixed

ah i understand. it looks to have been a change to our custom theme that required this change for our specific case.
thanks yhager for your input. we really appreciate your support!

[setting status back to fixed]

elijah lynn’s picture

elijah lynn
Location Portland, Oregon
commented

Just wanted to say thanks for this fix! I was just going to live with it but must say that it was one of those nagging bugs that always bothered me. I load 3000X2000+ images into the crop window sometimes and it really looks ugly until they load.

Looking forward to the next stable with this change!

Status: Fixed » Closed (fixed)

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