Merge lp:~dobey/ubuntuone-client-gnome/list-expand into lp:ubuntuone-client-gnome

Proposed by dobey
Status: Merged
Approved by: dobey
Approved revision: 4
Merged at revision: 4
Proposed branch: lp:~dobey/ubuntuone-client-gnome/list-expand
Merge into: lp:ubuntuone-client-gnome
Diff against target: 46 lines (+11/-12)
1 file modified
nautilus/share-dialog.c (+11/-12)
To merge this branch: bzr merge lp:~dobey/ubuntuone-client-gnome/list-expand
Reviewer Review Type Date Requested Status
Manuel de la Peña (community) Approve
Review via email: mp+76467@code.launchpad.net

Commit message

Be more explicit about expanding since GTK+ 3.x broke this

To post a comment you must log in.
Revision history for this message
Manuel de la Peña (mandel) wrote :

Would be nice to rename the var fro table to vbox, but is not an issue therefore I approve it and let dobey decide.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'nautilus/share-dialog.c'
--- nautilus/share-dialog.c 2011-07-20 20:44:39 +0000
+++ nautilus/share-dialog.c 2011-09-21 19:28:16 +0000
@@ -116,31 +116,30 @@
116 (_("Share")), GTK_RESPONSE_ACCEPT,116 (_("Share")), GTK_RESPONSE_ACCEPT,
117 NULL);117 NULL);
118 gtk_dialog_set_default_response (GTK_DIALOG (dialog), GTK_RESPONSE_ACCEPT);118 gtk_dialog_set_default_response (GTK_DIALOG (dialog), GTK_RESPONSE_ACCEPT);
119 gtk_dialog_set_response_sensitive (GTK_DIALOG (dialog), GTK_RESPONSE_ACCEPT, FALSE);119 gtk_dialog_set_response_sensitive (GTK_DIALOG (dialog),
120 GTK_RESPONSE_ACCEPT, FALSE);
120 gtk_window_set_icon_name (GTK_WINDOW (dialog), "ubuntuone");121 gtk_window_set_icon_name (GTK_WINDOW (dialog), "ubuntuone");
121 g_signal_connect (G_OBJECT (dialog), "response",122 g_signal_connect (G_OBJECT (dialog), "response",
122 G_CALLBACK (dialog_response_cb), NULL);123 G_CALLBACK (dialog_response_cb), NULL);
123124
124 area = gtk_dialog_get_content_area (GTK_DIALOG (dialog));125 area = gtk_dialog_get_content_area (GTK_DIALOG (dialog));
125126
126 table = gtk_table_new (3, 2, FALSE);127 table = gtk_vbox_new (FALSE, 12);
127 gtk_table_set_row_spacings (GTK_TABLE (table), 12);128 gtk_container_set_border_width (GTK_CONTAINER (table), 7);
128 gtk_table_set_col_spacings (GTK_TABLE (table), 6);129 gtk_box_pack_start (GTK_BOX (area), table, TRUE, TRUE, 0);
129 gtk_container_set_border_width (GTK_CONTAINER (table), 7);130 gtk_widget_show (table);
130 gtk_widget_show (table);
131 gtk_container_add (GTK_CONTAINER (area), table);
132131
133 dialog->user_picker = u1_contacts_picker_new ();132 dialog->user_picker = u1_contacts_picker_new ();
134 g_signal_connect (G_OBJECT (dialog->user_picker), "selection-changed",133 g_signal_connect (G_OBJECT (dialog->user_picker), "selection-changed",
135 G_CALLBACK (picker_selection_changed_cb), dialog);134 G_CALLBACK (picker_selection_changed_cb), dialog);
136 gtk_table_attach (GTK_TABLE (table), dialog->user_picker, 0, 2, 0, 2,135 gtk_box_pack_start (GTK_BOX (table), dialog->user_picker, TRUE, TRUE, 0);
137 GTK_FILL | GTK_EXPAND | GTK_SHRINK,
138 GTK_FILL | GTK_EXPAND | GTK_SHRINK, 3, 3);
139 gtk_widget_show (dialog->user_picker);136 gtk_widget_show (dialog->user_picker);
140137
141 dialog->allow_mods = gtk_check_button_new_with_mnemonic (_("_Allow Modification"));138 dialog->allow_mods = gtk_check_button_new_with_mnemonic (_("_Allow Modification"));
142 gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (dialog->allow_mods), TRUE); /* Default to RW */139 /* Default to RW */
143 gtk_table_attach (GTK_TABLE (table), dialog->allow_mods, 0, 2, 2, 3, GTK_FILL, GTK_FILL, 3, 3);140 gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (dialog->allow_mods),
141 TRUE);
142 gtk_box_pack_end (GTK_BOX (table), dialog->allow_mods, FALSE, FALSE, 0);
144 gtk_widget_show (dialog->allow_mods);143 gtk_widget_show (dialog->allow_mods);
145144
146 gtk_widget_set_size_request (GTK_WIDGET (dialog), 500, 450);145 gtk_widget_set_size_request (GTK_WIDGET (dialog), 500, 450);

Subscribers

People subscribed via source and target branches

to all changes: