I don't know how to replicate this over different content types but every now and then this happens consistently for my imagefield_crop filefield on specific content types. The crop window overlays directly on top of the preview window. I have attached a screenshot of before and after the patch.

With a few people's help at Do It With Drupal in New Orleans we have identified the issue (TZO-Scott & Ben) and created a patch (Webchick/Angie Byron) that fixes it. I am not sure if this is the proper way to do it but this fixes the issue. The style class (.widget-edit) is originally called for in the filefield.css but I patched the imagefield_crop.css instead.

CommentFileSizeAuthor
#2 imagefield_crop-crop-overlay-before-patch-01.PNG461.09 KBelijah lynn
#2 imagefield_crop-crop-overlay-after-patch-01.PNG567.39 KBelijah lynn
imagefield_crop-crop-overlay-01.patch581 byteselijah lynn

Comments

elijah lynn’s picture

elijah lynn
Location Portland, Oregon
commented
Status: Active » Needs review
elijah lynn’s picture

elijah lynn
Location Portland, Oregon
commented
StatusFileSize
new imagefield_crop-crop-overlay-after-patch-01.PNG567.39 KB
new imagefield_crop-crop-overlay-before-patch-01.PNG461.09 KB

Adding the images and changed status.

Bilmar’s picture

Bilmar commented

subscribing

elijah lynn’s picture

elijah lynn
Location Portland, Oregon
commented

Crosslinking - Duplicate Issue http://drupal.org/node/490416

OneTwoTait’s picture

OneTwoTait commented

subscribing

bleen’s picture

bleen commented

I applied the patch in the original post (manually because it was failing when I used "patch") and it works great.

Its worth noting that when I open my imagefield_crop.css file (from a fresh download) it is completely empty except for the CVS msg.

In any event, given that this is a CSS change I'm not worried about any code breakage, and when I add 

.widget-edit {
  clear: both;
}

to my css file then this problem goes away.

elijah lynn’s picture

elijah lynn
Location Portland, Oregon
commented

Maybe that was because I patched the dev version?

bleen’s picture

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

Indeed ... patch works fine on dev

rburgundy’s picture

rburgundy commented

tested and looks great. can we see this committed to dev version sometime soon? thank you!

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.