“Cannot set property deltaX of #<WheelEvent> which has only a getter” with Chrome 43 (Beta) #2751

New Issue

Closed

opened 2015-04-20 17:02:15 +00:00 by wjt · 2 comments

wjt commented

2015-04-20 17:02:15 +00:00

Using Google Chrome 43.0.2357.18 (Google's package, from the beta channel) on Fedora 21, using the mouse wheel (or two-finger scrolling on the touchpad) gives the following error, many many times:

Uncaught TypeError: Cannot set property deltaX of #<WheelEvent> which has only a getter

(anonymous function)	@	Ox.js?4902:274
Ox.forEach	@	Ox.js?4902:88
(anonymous function)	@	Ox.js?4902:274
Ox.forEach	@	Ox.js?4902:89
Ox.extend	@	Ox.js?4902:274
onMousewheel	@	UI.js?0.1.3665:269
f.event.dispatch	@	jquery.js?0.1.3665:3
handler	@	jquery.mousewheel.js?0.1.3665:81

Scrolling works fine in spite of this.

(Sorry for filling the pad.ma error log with this; I was trying to read the error log for some other problem.)

On the same machine, Chrome 42.0.2311.90 from the stable channel doesn't have this problem.

Using Google Chrome 43.0.2357.18 (Google's package, from the beta channel) on Fedora 21, using the mouse wheel (or two-finger scrolling on the touchpad) gives the following error, many many times: ``` Uncaught TypeError: Cannot set property deltaX of #<WheelEvent> which has only a getter (anonymous function) @ Ox.js?4902:274 Ox.forEach @ Ox.js?4902:88 (anonymous function) @ Ox.js?4902:274 Ox.forEach @ Ox.js?4902:89 Ox.extend @ Ox.js?4902:274 onMousewheel @ UI.js?0.1.3665:269 f.event.dispatch @ jquery.js?0.1.3665:3 handler @ jquery.mousewheel.js?0.1.3665:81 ``` Scrolling works fine in spite of this. (Sorry for filling the pad.ma error log with this; I was trying to read the error log for some other problem.) On the same machine, Chrome 42.0.2311.90 from the stable channel doesn't have this problem.

wjt added the

oxjs

label 2015-04-20 17:02:15 +00:00

wjt added this to the 14.04 milestone 2015-04-20 17:02:15 +00:00

rlx was assigned by wjt

2015-04-20 17:02:15 +00:00

wjt added the

normal

defect

labels 2015-04-20 17:02:15 +00:00

wjt commented

2015-05-13 15:54:31 +00:00

Author

Attachment 0001-Element-fix-mousewheel-event-before-modifying-it.patch (1524 bytes) added

Element: 'fix' mousewheel event before modifying it

**Attachment** 0001-Element-fix-mousewheel-event-before-modifying-it.patch (1524 bytes) added Element: 'fix' mousewheel event before modifying it

0001-Element-fix-mousewheel-event-before-modifying-it.patch

1.5 KiB

wjt commented

2015-05-20 10:36:13 +00:00

Author

In []changeset:oxjs,3693:

#!CommitTicketReference repository="" revision="oxjs,3693"
Element: 'fix' mousewheel event before modifying it

In strict mode (in recent browsers), modifying deltaX on a WheelEvent
throws an exception:

Uncaught TypeError: Cannot set property deltaX of #<WheelEvent> which has only a getter

In non-strict mode or older browsers, assigning to deltaX is just a
no-op.

https://github.com/jquery/jquery-mousewheel/blob/master/jquery.mousewheel.js
is cited in this function; it uses $.event.fix() to make a mutable
jQuery.Event-flavoured copy of the native event before modifying its
properties. Let's do the same.

Fixes #2751

In []changeset:oxjs,3693: ``` #!CommitTicketReference repository="" revision="oxjs,3693" Element: 'fix' mousewheel event before modifying it In strict mode (in recent browsers), modifying deltaX on a WheelEvent throws an exception: Uncaught TypeError: Cannot set property deltaX of #<WheelEvent> which has only a getter In non-strict mode or older browsers, assigning to deltaX is just a no-op. https://github.com/jquery/jquery-mousewheel/blob/master/jquery.mousewheel.js is cited in this function; it uses $.event.fix() to make a mutable jQuery.Event-flavoured copy of the native event before modifying its properties. Let's do the same. Fixes #2751 ```

wjt added the

fixed

label 2015-05-20 10:36:13 +00:00

wjt closed this issue

2015-05-20 10:36:13 +00:00