Strange issue in Chrome where the page jumps around while the Crop interface is open (works fine in IE11 and Firefox)

To replicate:

  1. Upload image
  2. Click anywhere else on the page
  3. Click anywhere in the image being cropped
  4. Page jumps

I've disabled jQuery update, used the default Bartik theme and I've also updated to the latest jCrop version as well yet the issue still occurs.

CommentFileSizeAuthor
#16 simplecrop-2422583-16.patch245 bytespetu
#14 simplecrop-2422583-13.patch244 bytespetu
#13 simplecrop-page-jumoing-2422583-13.patch470 bytespetu
#2 simplecrop chrome.mp42.54 MBtunaman

Comments

spleshka’s picture

spleshka
he/him
Primary language English
Location UAE
commented
Status: Active » Postponed (maintainer needs more info)

That is very strange issue, I couldn't reproduce it in Chrome. Could you please install the module on simplyest.me and tell me if you still have this problem?

tunaman’s picture

tunaman commented
StatusFileSize
new simplecrop chrome.mp42.54 MB

Hi Spleshka,

Issue replicated on Simplytestme. I forgot to mention, after uploading the image, do not apply a crop, click elsewhere on the screen and then click on the image.

Screen capture attached.

tunaman’s picture

tunaman commented
Status: Postponed (maintainer needs more info) » Active
spleshka’s picture

spleshka
he/him
Primary language English
Location UAE
commented

Thanks for your video, now I see what you mean. Unfortunately, I still can't reproduce it in Chrome, even if I install module on simplytest.me and follow your steps. What OS and Chrome version do you use?

tunaman’s picture

tunaman commented

I've tested in:

  • Windows 8, Version 40 Chrome.
  • Windows 7, Version 40 Chrome.
spleshka’s picture

spleshka
he/him
Primary language English
Location UAE
commented

Unbelievable, I've tested in Windows 7, Chrome 40 with no success. I would really like to solve this issue, but have no idea how to reproduce it. Could you open a "Anonymous mode" in Chmore and reproduce this issue? Probably, one of your Chrome plugins leads to this issue.

tunaman’s picture

tunaman commented

Same problem with no extensions enabled or installed in Chrome I'm afraid :(

spleshka’s picture

spleshka
he/him
Primary language English
Location UAE
commented

I've asked other devs to reproduce this issue and they couldn't. Do you have any idea about our next steps? I did my best to reproduce this issue with no success. Probably, it is specific to your Chrome. If you will have any new information about this issue, please let me know.

iampuma’s picture

iampuma
Primary language Dutch
Location Frankfurt am Main, Germany
commented

Due to the theme I was using I had the same problem. The following CSS snippet mentioned in this comment on the jcrop Github project solved it though.

.jcrop-holder>input {
    top: 0 !important;
}
spleshka’s picture

spleshka
he/him
Primary language English
Location UAE
commented

Thanks @iampuma, that could be the trick! @tunaman, does this workaround solves your issue?

tunaman’s picture

tunaman commented

Seems to have done the trick! Thanks guys.

tunaman’s picture

tunaman commented
Status: Active » Closed (fixed)
petu’s picture

petu commented
Version: 7.x-1.x-dev » 7.x-1.8
StatusFileSize
new simplecrop-page-jumoing-2422583-13.patch470 bytes

#9 helped me. Thank you, @iampuma!

The patch is enlosed.

petu’s picture

petu commented
StatusFileSize
new simplecrop-2422583-13.patch244 bytes

The patch was improved.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Version: 7.x-1.8 » 7.x-1.x-dev
Status: Closed (fixed) » Needs work

Thanks Petu!

The original proposal used a dot in front: ".jcrop-holder>input {", but in your patch you use "jcrop-holder>input {". It looks like that would not be valid and you need a class selector? Maybe I'm missing something.

petu’s picture

petu commented
StatusFileSize
new simplecrop-2422583-16.patch245 bytes

Thanks @klausi, sure it's a typo!
Fixed.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs work » Reviewed & tested by the community

Thanks, looks good to me!

  • klausi committed ebf5690a on 7.x-1.x authored by petu 
    fix(css): Fix page jumping in Chrome (#2422583 by iampuma, petu)
klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Reviewed & tested by the community » Fixed

Merged, thanks!

Status: Fixed » Closed (fixed)

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