Merge lp:~mrasmus/indicator-session/fix-597317 into lp:indicator-session/0.1

Proposed by Matthew Rasmus on 2011-02-07
Status: Merged
Merged at revision: 136
Proposed branch: lp:~mrasmus/indicator-session/fix-597317
Merge into: lp:indicator-session/0.1
Diff against target: 24 lines (+15/-0)
1 file modified
src/dialog.c (+15/-0)
To merge this branch: bzr merge lp:~mrasmus/indicator-session/fix-597317
Reviewer Review Type Date Requested Status
Ted Gould (community) 2011-02-07 Approve on 2011-02-09
Review via email: mp+48746@code.launchpad.net
To post a comment you must log in.
Ted Gould (ted) wrote :

  review approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/dialog.c'
2--- src/dialog.c 2010-03-29 19:08:16 +0000
3+++ src/dialog.c 2011-02-07 02:37:53 +0000
4@@ -226,5 +226,20 @@
5
6 gtk_dialog_set_default_response(GTK_DIALOG(dialog), GTK_RESPONSE_OK);
7
8+ /* The following is a workaround to fix an issue in GtkMessageDialog
9+ in which the user can tab through the text in addition to
10+ the buttons. */
11+ GtkWidget *message_area = gtk_message_dialog_get_message_area(GTK_MESSAGE_DIALOG(dialog));
12+ GList *children = gtk_container_get_children(GTK_CONTAINER(message_area));
13+ GList *l;
14+
15+ for (l = children; l != NULL; l = g_list_next (l))
16+ {
17+ GtkWidget *child = l->data;
18+ gtk_widget_set_can_focus(child, FALSE);
19+ }
20+
21+ g_list_free (children);
22+
23 return dialog;
24 }

Subscribers

People subscribed via source and target branches