Merge lp:~jassmith/go-home-applet/atk into lp:go-home-applet

Proposed by Jason Smith
Status: Merged
Merged at revision: not available
Proposed branch: lp:~jassmith/go-home-applet/atk
Merge into: lp:go-home-applet
Diff against target: None lines
To merge this branch: bzr merge lp:~jassmith/go-home-applet/atk
Reviewer Review Type Date Requested Status
David Barth (community) Approve
Mirco Müller (community) Abstain
Eitan Isaacson Pending
Review via email: mp+10741@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Jason Smith (jassmith) wrote :

Makes a11y work with the proper orca config, however this really will need feedback from the a11y community as I am not a user with these needs.

lp:~jassmith/go-home-applet/atk updated
31. By Jason Smith <jason@t500>

translatable strings

Revision history for this message
Mirco Müller (macslow) wrote :

Good, translatable strings are in. Still regarding general use of ATK I'd rather see Eitan "eeejay" Isaacson comment on that.

review: Abstain
Revision history for this message
David Barth (dbarth) wrote :

Same as the other ATK patch. Go ahead.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/applet.c'
2--- src/applet.c 2009-02-16 12:27:20 +0000
3+++ src/applet.c 2009-08-26 14:11:40 +0000
4@@ -321,6 +321,7 @@
5 WnckScreen *screen;
6 GoHomeApp *app;
7 GtkWidget *eb;
8+ AtkObject *atk;
9 gchar *key;
10
11 if (strcmp (iid, "OAFIID:GNOME_GoHome") != 0)
12@@ -373,6 +374,11 @@
13
14 gtk_widget_show_all (GTK_WIDGET (applet));
15
16+ atk = gtk_widget_get_accessible (app->image);
17+ atk_object_set_name (atk, "Go Home");
18+ atk_object_set_description (atk, "Go home, show the home window.");
19+ atk_object_set_role (atk, ATK_ROLE_PUSH_BUTTON);
20+
21 /* Signals */
22 g_signal_connect (applet, "change-background",
23 G_CALLBACK (cw_panel_background_changed), NULL);

Subscribers

People subscribed via source and target branches

to all changes: