After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 509068 - We should prevent the user from arrowing into combo boxes in FF
We should prevent the user from arrowing into combo boxes in FF
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.21.x
Other All
: Normal normal
: 2.22.0
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks: 404403
 
 
Reported: 2008-01-13 03:13 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2008-07-22 19:33 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
revision 1 (4.66 KB, patch)
2008-01-13 03:15 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2008-01-13 03:13:04 UTC
It is still possible to arrow into a combo box if the circumstances are just right.  At this point setCaretPosition() grabs focus on the combo box.  While we have some (and are working on other) "escape mechanisms", I am of the opinion that we just shouldn't allow users to arrow into a combo box period. :-)

My suggestion is that we treat combo boxes much the same way that we do lists in forms, namely: If we were about to setCaretPosition() on a combo box, instead set it to the caret offset within the parent that immediately precedes it.  The announcement of the combo box should alert the user that they can do something (e.g. tab into it).

Tabbing and using structural navigation to move among form fields would still grab focus (just like with lists).

Finally, we already have Alt+Down Arrow to expand a combo box if a menu item somehow was arrowed into.  This handler should also check to see if we're at a combo box (but not in a menu item).

Patch to follow.
Comment 1 Joanmarie Diggs (IRC: joanie) 2008-01-13 03:15:11 UTC
Created attachment 102705 [details] [review]
revision 1

This patch does what I proposed in the opening report.

Mike please test.  Thanks!
Comment 2 Mike Pedersen 2008-01-13 04:33:49 UTC
I think this is the correct approach.  This should always keep users from ever getting stuck in comboboxes when they didn't intend to be their.  
Comment 3 Joanmarie Diggs (IRC: joanie) 2008-01-13 04:46:56 UTC
That's the idea anyway. <smile>

Will, please review.
Comment 4 Willie Walker 2008-01-13 23:33:21 UTC
Looks good to me.  Thanks, and please commit assuming you've tested this and it pylints to a 10.00.  :-)  

PS - I wonder if some day we'll end up doing the same for text areas.
Comment 5 Joanmarie Diggs (IRC: joanie) 2008-01-14 16:11:29 UTC
Patch committed.  Moving to pending.
Comment 6 Joanmarie Diggs (IRC: joanie) 2008-01-17 03:17:27 UTC
Closing this one as FIXED as 2.21.5 has come and gone. It should not introduce regressions, but if it does/has, we can reopen or file a new bug.