Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#12700 closed defect (fixed)

[patch] [cla] RangeSlider controls (mouse + keyboard) to affect 'focused' node

Reported by: Jonathan Bond-Caron Owned by: dante
Priority: high Milestone: 1.7
Component: DojoX Form Version: 1.6.0
Keywords: Cc:
Blocked By: Blocking:

Description

The behavior of the 'RangeSlider?' isn't intuitive.

The "left" and "right" buttons should move the currently focused node.

Attachments (2)

FocusSlide.patch​ (9.7 KB) - added by Jonathan Bond-Caron 8 years ago.
Allow moving focused 'cursor node'
12700.patch​ (5.0 KB) - added by Katie Vance 8 years ago.
Refresh previous patch- add HOME/END, remove _curFocus

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by Jonathan Bond-Caron

Attachment: FocusSlide.patch​ added

Allow moving focused 'cursor node'

comment:1 Changed 8 years ago by Jonathan Bond-Caron

This patch requires ticket #12701

I'm not sure if it's the best approach but in my opinion, it simplifies event handling (not having to think about LTR/RTL).

comment:2 Changed 8 years ago by bill

Summary: CLA: RangeSlider controls (mouse + keyboard) to affect 'focused' node → [patch] [cla] RangeSlider controls (mouse + keyboard) to affect 'focused' node

comment:3 Changed 8 years ago by Katie Vance

12700.patch removes the dependency on #12701. I also added support for HOME/END, removed use of private _curFocus, and cleaned up some of the code.

Changed 8 years ago by Katie Vance

Attachment: 12700.patch​ added

Refresh previous patch- add HOME/END, remove _curFocus

comment:4 Changed 8 years ago by Douglas Hays

Resolution: → fixed
Status: new → closed

In [26284]:

Fixes #12700. Add HOME and END support and fix other keyboard problems. Removed use of private _curFocus

comment:5 Changed 8 years ago by Douglas Hays

Milestone: tbd → 1.7
Note: See TracTickets for help on using tickets.