I'm seeing this error in the browser console, after uploading an image, or when editing a node that already has an image.

Also, when editing, the crosshair is always at (0,0): it doesn't locate on the saved coordinates.

CommentFileSizeAuthor
#6 drupalbehaviorerror-2537110-6.patch1004 bytesbleen
#1 focal_point-2537110.patch436 bytesjonhattan

Comments

jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
commented
Status: Active » Needs review
StatusFileSize
new focal_point-2537110.patch436 bytes

This fixes the error message, but the crosshair is still at (0,0)

bleen’s picture

bleen commented

When try on simplytest.me the crosshair is always in the correct place. Are you sure you don't have some other JS errors?

jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
commented

I'm sorry, effectively the crosshair is in place with this patch applied.

bleen’s picture

bleen commented

I have not been able to reproduce the JS error reported in the original post. A quick chat with @jonhattan says:

[11:07:15]  <bleen>	jonhattan: I'm not seeing a js error with or without your patch
[11:07:23]  <bleen>	jonhattan: browser & version?
[11:08:44]  <jonhattan>	bleen, firefox 39.0
[11:09:36]  <bleen>	jonhattan: OS?
[11:10:05]  <jonhattan>	bleen, Debian
[11:10:12]  <jonhattan>	indeed it is iceweasel not firefox
[11:10:19]  <bleen>	hrmmm
[11:10:35]  <jonhattan>	bleen, neither see any further error with  Chromium Version 43.0.2357.65 Built on 8.0, running on Debian 8.1 (64-bit)
[11:11:10]  <bleen>	jonhattan: ok, good info.  I'm going to ask someone better than I am with JS to take a look at your patch
jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
commented

To clarify further:

I'm getting the DrupalBehaviorError: attach ; focalPointIndicator: event is not defined error only with Iceweasel 39.0 on Debian 8 (iceweasel is debian's relabel of firefox). The patch in #1 fixes it.

I'm not getting the error with Chromium Version 43.0.2357.65 Built on 8.0, running on Debian 8.1 (64-bit). It works with and without the patch in #1.

bleen’s picture

bleen commented
StatusFileSize
new drupalbehaviorerror-2537110-6.patch1004 bytes

Can you test this instead...

I'm told its considered better practice to use the "on" method

jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
commented

It works

bleen’s picture

bleen commented
Status: Needs review » Fixed

  • bleen18 committed 17c6b07 on 8.x-1.x 
    Issue #2537110 by jonhattan, bleen18: Fixed DrupalBehaviorError: attach...

Status: Fixed » Closed (fixed)

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