New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Image Focus doesn't go to where mouse is clicked #1976

Closed
TomS- opened this Issue May 29, 2018 · 6 comments

Comments

Projects
None yet
3 participants
@TomS-

TomS- commented May 29, 2018

Describe the bug
On Chrome, I haven't tested any other browsers. When setting the focus point on an image, the alignment circle does not match where you've clicked. I've found the further to the right or bottom you click, the bigger the distance.

To Reproduce
Steps to reproduce the behavior:

  1. Create an assets field
  2. Select an image
  3. Click to set the focus point
  4. Notice that it does not align with where you clicked
  5. Click further to the right to see it have even less accuracy.

Expected behavior
The center of the focus point should be where you clicked.

Environment details (please complete the following information):

  • Statamic Version 2.9.4
@jasonvarga

This comment has been minimized.

Member

jasonvarga commented May 29, 2018

Works fine in Chrome for me. Can you give any more details? Does it do it for all images?

@TomS-

This comment has been minimized.

TomS- commented May 29, 2018

Might be worth mentioning that the images are inside replicators. That may have something to do with it.

@TomS-

This comment has been minimized.

TomS- commented May 31, 2018

The issue is with portrait images. Portrait images are not accurate when setting focus point.

@jasonvarga

This comment has been minimized.

Member

jasonvarga commented Jun 1, 2018

Are you sure? Can you provide any more details? The image?

@TomS-

This comment has been minimized.

TomS- commented Jun 7, 2018

The width of the image needs to be smaller than 330px - I resize the images to exact size before uploading not to fill up the server.

@jackmcdade jackmcdade added bug and removed can't recreate labels Jun 7, 2018

@jackmcdade

This comment has been minimized.

Member

jackmcdade commented Jun 7, 2018

Ah, there it is. It's the width.

@jackmcdade jackmcdade self-assigned this Jun 7, 2018

@jackmcdade jackmcdade closed this Jun 7, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment