Opened 11 years ago

Closed 11 years ago

Last modified 8 years ago

#6747 closed enhancement (fixed)

[patch] [cla] Slider/Spinner: Safari mousewheel support

Reported by: dante Owned by: Douglas Hays
Priority: high Milestone: 1.2
Component: Dijit - Form Version: 1.1.0
Keywords: Cc:
Blocked By: Blocking:

Description

I've been playing around with mousewheel support and came up with a really nifty mousewheel listener to reliably give a positive or negative delta value. (actually in #6485, but this applies it to Slider)

It works w/ Safari 3, FF2/3, IE7 (from within macos/parallels, so need testing w/ "real" windows mouse), and Opera (though slider test locks up in opera 9, my demo using this techmo doesn't).

This is similar to #6485 in the shorthand and connections. I plan to go back and test spinner in Safari/Opera? and supply new patch for that later.

Attachments (1)

slider.patch (3.3 KB) - added by dante 11 years ago.
adds opera and safari mousewheel support to _spinner and slider

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by dante

Attachment: slider.patch added

adds opera and safari mousewheel support to _spinner and slider

comment:1 Changed 11 years ago by bill

Owner: changed from bill to Douglas Hays
Summary: [dijit.form] Safari mousewheel support in Slider[patch] [cla] Slider/Spinner: Safari mousewheel support

Looks good to me.

comment:2 Changed 11 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

(In [13739]) Fixes #6747 #6176. Committing patch from dante to add safari mouse wheel event handling.

comment:3 Changed 8 years ago by bill

Component: DijitDijit - Form
Note: See TracTickets for help on using tickets.