Opened 10 years ago

Closed 7 years ago

#10064 closed defect (patchwelcome)

Grid: Keyboard scrolling through header doesn't always work correctly

Reported by: Becky Gibson Owned by: Evan
Priority: high Milestone: future
Component: DojoX Grid Version: 1.3.2
Keywords: a11y Cc:
Blocked By: Blocking:

Description

load http://archive.dojotoolkit.org/dojo-2009-10-08/dojotoolkit/dojox/grid/tests/test_keyboard.html.

Press the tab key to put the focus indicator into the column 1 column header. Press the right arrow to move the focus indicator to column 2, column 3, column 4 headers. With the focus indicator in column 4, press the right arrow. The focus indicator goes to column 5 but it does not scroll completely into view. If you press the right arrow key again, the focus indicator goes to column 6 and the grid scrolls so that column 6 is fully in view. This only seems to be an issue in Firefox. In Safari 4 on OS X and IE 8 on windows scrolling works correctly.

It may be an issue with _FocusManger.js _scrollInfo() or _scrollHeaders().

This ticket is a refinement of #10039

Attachments (1)

10064.patch (608 bytes) - added by haysmark 7 years ago.
FF grid header scrolling fix.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 10 years ago by Nathan Toone

Milestone: tbd1.5

comment:2 Changed 9 years ago by Nathan Toone

Milestone: 1.5future
Owner: Nathan Toone deleted

Moving my tickets to future, as I am not currently working on them.

Changed 7 years ago by haysmark

Attachment: 10064.patch added

FF grid header scrolling fix.

comment:3 Changed 7 years ago by haysmark

Owner: set to Evan
Status: newassigned

Please pass on to Oliver. For some reason scrollLeft is not updated the first time it is set. I attached a very strange fix.

comment:4 Changed 7 years ago by bill

DojoX Grid and EnhancedGrid are deprecated in favor of dgrid and gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.

comment:5 Changed 7 years ago by bill

Resolution: patchwelcome
Status: assignedclosed
Note: See TracTickets for help on using tickets.