ColorPalette: Focus set on doc not on cell

[ptbrunet]

Go to .../dojotoolkit/dijit/tests/test_ColorPalette.html and set focus on the first cell, press ctrl tab twice to move focus away and back. The focus moves back to the first cell. Now move focus to some other cell and do the same thing, focus doesn't move to the orignal cell but instead to the document.

[ptbrunet]

This might be related: Go to the button test and put focus on one of the drop down menus (including the palette dropdown), press ctrl Tab twice, no focus is fired and pressing tab doesn't do anything.

[ptbrunet]

Ctrl Tab moves between tabs. I should have said Alt Tab in my opening comment - but the same issue exists with Ctrl Tab.

[simonjb]

I've been having a look at this. The ColorPalette dynamically sets tabindex="0" on the img cell with focus, rather than setting tabindex="0" on the ColorPalette widget and redirecting focus with focus() (as the menu and toolbar do). Moving to the menu and toolbar model will fix this bug. However, Firefox cannot focus() img elements with tabindex="-1". I have filed a bug against Mozilla:

​https://bugzilla.mozilla.org/show_bug.cgi?id=417760

Fixing the Mozilla bug won't really help us though as FF2 would not be fixed (I'm thinking). So, a possible solution would be to modify the ColorPalette such that the cell imgs are wrapped in a div, and focus the div.

[simonjb]

Or, if it does get fixed in FF3 we could just live with the bug in FF2.

[Becky Gibson]

I could live with this bug in FF2 as long as the color palette is still keyboard accessible, of course :-). Or, if we wrap the img elements with another element, could it be a span rather than a div since divs are now getting put in the a11y hierarchy? I didn't look at the code so there is probably some requirement (like sizing) for using a div).

[simonjb]

(In [12831]) Wrap the ColorPalette? imgs in spans as Firefox cannot programmatically focus imgs with tabindex=-1; fixes #5119

[bill]

(In [20780]) Big cleanup of ColorPalette code, fixing focus issues and rolling back most of #5119 as it missed the point that we should set tabIndex >= 0 on nodes that we focus. Fixes #10315, refs #5119 !strict.