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 154604 - gok should not always present multiple actions to the user
gok should not always present multiple actions to the user
Status: RESOLVED FIXED
Product: gok
Classification: Deprecated
Component: general
unspecified
Other Linux
: High major
: ---
Assigned To: David Bolter
David Bolter
AP2
Depends on:
Blocks: 154566
 
 
Reported: 2004-10-05 19:52 UTC by David Bolter
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
special cases for widgets with three actions: click, press, and release (1.87 KB, patch)
2004-10-05 19:55 UTC, David Bolter
none Details | Review
improved patch (1.79 KB, patch)
2004-10-06 20:39 UTC, David Bolter
needs-work Details | Review
proposed patch (1.97 KB, patch)
2004-10-13 16:03 UTC, David Bolter
none Details | Review
updated patch against head (1.94 KB, patch)
2004-10-26 18:24 UTC, David Bolter
committed Details | Review

Description David Bolter 2004-10-05 19:52:31 UTC
Activate the panel with the launcher, 
select UIGrab, 
select Launch
result: user is presented with click, press, and release.

Better: just invoke the click.

Patch coming.
Comment 1 David Bolter 2004-10-05 19:55:42 UTC
Created attachment 32261 [details] [review]
special cases for widgets with three actions: click, press, and release

Bill, this patch needs to be reviewed and tested.  I'll be in tomorrow
(Wednesday)
Comment 2 David Bolter 2004-10-06 20:39:28 UTC
Created attachment 32305 [details] [review]
improved patch
Comment 3 bill.haneman 2004-10-12 13:32:58 UTC
test/QA has suggested that this bug should be treated as P3 or P2.
Comment 4 bill.haneman 2004-10-12 13:36:57 UTC
David, as I said on IRC, the right place to put the reduction logic is when
determining the number of actions, not the number of actionable children:

i.e. branch if n_actionable_children > 0 && has_multi_useful_actions ()

or something like that, where has_multi_useful_actions (action) is logic
much like your three_actions_reduce(), except reversed in the sense of the
boolean return. 
Comment 5 David Bolter 2004-10-13 16:03:30 UTC
Created attachment 32564 [details] [review]
proposed patch

This patch has been tested to solve the click, press, release reduction, but
has not been tested with counter cases.  Please test and review.
Comment 6 David Bolter 2004-10-19 15:47:05 UTC
Counter cases would be components that expose multiple actions other than the:
click, press, release trio.  Got any?
Comment 7 Calum Benson 2004-10-21 16:47:28 UTC
Apologies for spam-- ensuring Sun a11y team are cc'ed on all current a11y bugs.
 Filter on "SUN A11Y SPAM" to ignore.
Comment 8 David Bolter 2004-10-26 18:24:37 UTC
Created attachment 33083 [details] [review]
updated patch against head
Comment 9 bill.haneman 2004-10-26 18:38:43 UTC
raising severity because this is an efficiency killer.
Comment 10 David Bolter 2004-10-27 16:19:53 UTC
Fixed in CVS.

(Note: we'll need to keep an eye out for situations where this reduction might
not be desirable)
Comment 11 Harry Lu 2004-11-04 03:27:19 UTC
I have some concerns here. As I can see from some codes, not everyone connect to
the "clicked" signal of a gtkbutton. Some just connect to the "pressed" signal,
and in some cases, they connect to "button_press_event" signal.  So their signal
handling function won't be called when you just let the button emit a "clicked"
signal. 
In gailbutton.c, the action "press" will simulate a button press event. So their
function can be called when doing the action "press".

So I think leave the 3 actions there, user can try "click" first, if this
doesn't work, he might then try "press".

Or maybe GOK could have a option to let user choose whether showing all actions.
Comment 12 David Bolter 2004-11-04 14:06:21 UTC
I think having a user option is a good idea.  

Ideally the first exposed action (at the first index) would be the primary
action for a given component but it sounds like that would require extra effort
from the application developer.  I don't like relying on this.

Bill, your thoughts?