Opened 14 years ago

Closed 14 years ago

#3945 closed defect (fixed)

add context menu to tab title to keyboard delete tabs

Reported by: Becky Gibson Owned by: Becky Gibson
Priority: low Milestone: 1.1
Component: Accessibility Version: 0.9
Keywords: Cc:
Blocked By: Blocking:


We added support for pressing the delete key with focus on a tab title in order to delete a tab. But, this isn't discoverable for someone who can not see the close icon in the tab title area. The DHTML style guide group also recommends adding a context menu to delete the tab. If the tab title is marked with haspopup=true this will be somewhat discoverable (altho not supported until FF 3 but won't hurt to put in now). Some concerns that this context menu is invoked via enter key or click making this act like a button but tab title needs to retain the role of "tab".

Only context menu entry is "delete". This also has i18n implications. I think delete or remove is the right word since close implies that it can be added back /opened again.

from DHTML Style Guide June 29, 2007 meeting minutes ( TW: options other than DELETE - CTRL+F4 a possibility, have

to get comments on that - what we are saying is context menu and keyboard shortcut combo fine - great if had property so didn't have to use context menu, but maybe that's phase 2

Attachments (1)

3945.patch (1.7 KB) - added by Becky Gibson 14 years ago.
add context menu - in progress

Download all attachments as: .zip

Change History (7)

comment:1 Changed 14 years ago by davidb

Priority: normallow

comment:2 Changed 14 years ago by davidb

Milestone: 1.01.1

comment:3 Changed 14 years ago by Becky Gibson

Owner: changed from davidb to Becky Gibson

comment:4 Changed 14 years ago by Becky Gibson

Status: newassigned

Changed 14 years ago by Becky Gibson

Attachment: 3945.patch added

add context menu - in progress

comment:5 Changed 14 years ago by Becky Gibson

(In [12280]) refs #3945 added close context menu to titles of closable tabs for a11y. Updated test files for easier screen reader testing. May need to modify changes after further discussion with Firefox 1 a11y developer. New "Close" string in nls/common.js !strict

comment:6 Changed 14 years ago by Becky Gibson

Resolution: fixed
Status: assignedclosed

[12280] actually closes this ticket and refs #5410

Note: See TracTickets for help on using tickets.