Ticket #6747 (closed enhancement: fixed)

Opened 2 months ago

Last modified 2 months ago

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

Reported by: dante Owned by: doughays
Priority: normal Milestone: 1.2
Component: Dijit Version: 1.1.0
Severity: normal Keywords:
Cc:

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

slider.patch (3.3 kB) - added by dante 2 months ago.
adds opera and safari mousewheel support to _spinner and slider

Change History

Changed 2 months ago by dante

adds opera and safari mousewheel support to _spinner and slider

Changed 2 months ago by bill

  • owner changed from bill to doughays
  • summary changed from [dijit.form] Safari mousewheel support in Slider to [patch] [cla] Slider/Spinner: Safari mousewheel support

Looks good to me.

Changed 2 months ago by doughays

  • status changed from new to closed
  • resolution set to fixed

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

Note: See TracTickets for help on using tickets.