Opened 12 years ago

Closed 12 years ago

Last modified 7 years ago

#3811 closed task (fixed)

implement keyboard a11y for tooltips

Reported by: Becky Gibson Owned by: simonjb
Priority: high Milestone: 1.0
Component: Accessibility Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by simonjb)

currently the tooltips on the test_tooltip.html page only display on mouse over - we need keyboard support for tooltips as well.

See Becky's Proposed A11y behavior for Tooltip.

Attachments (1)

Tooltip.diff (3.0 KB) - added by simonjb 12 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 12 years ago by simonjb

Owner: changed from Becky Gibson to simonjb

comment:2 Changed 12 years ago by simonjb

Description: modified (diff)
Status: newassigned

comment:3 Changed 12 years ago by simonjb

(In [10419]) Implemented keyboard a11y for tooltips. Refs #3811. This is a simple solution and is not without problems. With this commit it is now possible to close a tooltip without an event going to the tooltip. For example, if tooltip A is opened on mouseover and if tooltip B is opened on focus while tooltip A is open, then tooltip A will close without an event going to tooltip A. This causes tooltip A's isShowingNow to be out of sync with reality with the effect that it may not open in the future (because it thinks that it is already open.)

Changed 12 years ago by simonjb

Attachment: Tooltip.diff added

comment:4 Changed 12 years ago by simonjb

Resolution: fixed
Status: assignedclosed

(In [10444]) Removed Tooltip.isShowingNow and moved the checks to see if we are re-opening an already open tooltip or if we are closing the right tooltip to _MasterTooltip. Fixes #3811.

comment:5 Changed 8 years ago by bill

(In [25493]) Remove code to keep a tooltip open if the associated element still has focus (even though the mouse moved away). Code was added in [10444] but it's unclear why it's needed. I suppose it's for when a user tabs into a field and then accidentally moves the mouse over it, but that doesn't seem important enough to worry about. Fixes #13225, refs #3811 !strict.

comment:6 Changed 7 years ago by bill

In [29201]:

Removing dead code which should have been removed in [25493], refs #3811, fixes #15668 !strict, thanks Karl.

Note: See TracTickets for help on using tickets.