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 132316 - terminal widget's context menu posting isn't exposed as an AtkAction
terminal widget's context menu posting isn't exposed as an AtkAction
Status: RESOLVED FIXED
Product: vte
Classification: Core
Component: general
0.11.x
Other All
: High normal
: ---
Assigned To: VTE Maintainers
VTE Maintainers
AP3
Depends on:
Blocks:
 
 
Reported: 2004-01-23 17:11 UTC by bill.haneman
Modified: 2007-01-22 20:01 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
Proposed patch (7.15 KB, patch)
2004-01-26 13:38 UTC, padraig.obriain
needs-work Details | Review
Corrected patch - deals with return value from popup-menu signal (7.18 KB, patch)
2004-01-28 17:16 UTC, padraig.obriain
none Details | Review

Description bill.haneman 2004-01-23 17:11:55 UTC
vte has a context menu when focussed, but there's no AtkAction exposed on
the focussable widget to pop it up.  there ought to be a 'menu' action, in
line with the solution for nautilus (see bug 132156).
Comment 1 padraig.obriain 2004-01-26 13:38:14 UTC
Created attachment 23761 [details] [review]
Proposed patch
Comment 2 padraig.obriain 2004-01-26 13:43:19 UTC
In addition to adding AtkAction this patch removes implementation for
atk_grab_focus as the implementation in gail_widget.c is better.
Comment 3 padraig.obriain 2004-01-28 17:16:18 UTC
Created attachment 23845 [details] [review]
Corrected patch - deals with return value from popup-menu signal
Comment 4 Kjartan Maraas 2004-10-18 10:12:12 UTC
Upping pri. Please review and commit if it's ok.
Comment 5 Calum Benson 2004-10-21 16:48:02 UTC
Apologies for spam-- ensuring Sun a11y team are cc'ed on all current a11y bugs.
 Filter on "SUN A11Y SPAM" to ignore.
Comment 6 Chris Wilson 2007-01-22 09:53:33 UTC
Ping?

The argument is sound and the patch looks sane. Commit?
Comment 7 bill.haneman 2007-01-22 10:01:54 UTC
As Will suggests, I think this patch could be dangerous to accessibility.  Please let's have some more discussion, I haven't had time to review it yet
Comment 8 Chris Wilson 2007-01-22 10:12:45 UTC
Just to check that I haven't confused you in a flurry of review requests... I presume comment #7 was actually addressed to bug 398006?
Comment 9 bill.haneman 2007-01-22 11:16:50 UTC
Hi Chris, thanks, yes.  Sincere apologies for the confusion!
Comment 10 Behdad Esfahbod 2007-01-22 17:46:52 UTC
No idea about this one.  If it does the job, feel free to commit it, but please make sure Rich tests it.
Comment 11 Chris Wilson 2007-01-22 18:30:16 UTC
r1496: 2007-01-22  Chris Wilson <chris@chris-wilson.co.uk>

	Bug 132316 – terminal widget's context menu posting isn't exposed as an AtkAction

	Original patch by <padraig.obriain@sun.com>

	* src/vteaccess.c: (vte_terminal_accessible_free_private_data),
	(vte_terminal_accessible_component_init),
	(vte_terminal_accessible_do_action),
	(vte_terminal_accessible_get_n_actions),
	(vte_terminal_accessible_action_get_description),
	(vte_terminal_accessible_action_get_name),
	(vte_terminal_accessible_action_get_keybinding),
	(vte_terminal_accessible_action_set_description),
	(vte_terminal_accessible_action_init),
	(vte_terminal_accessible_get_type):
		Remove grab_focus and defer to the parent's implementation.
		Add an AtkAction iface.

Comment 12 Joanmarie Diggs (IRC: joanie) 2007-01-22 20:01:48 UTC
Rich asked me to take a look at this one as well.  Having installed revision 1497, I'm not noticing any differences in Orca's behavior when navigating in gnome-terminal's context menu.  (Should I be?)