Merge lp:~psusi/ubuntu/lucid/nautilus/openwith into lp:ubuntu/lucid-proposed/nautilus

Proposed by Phillip Susi
Status: Merged
Merged at revision: 186
Proposed branch: lp:~psusi/ubuntu/lucid/nautilus/openwith
Merge into: lp:ubuntu/lucid-proposed/nautilus
Diff against target: 59 lines (+38/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/02_dont_set_default.patch (+29/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~psusi/ubuntu/lucid/nautilus/openwith
Reviewer Review Type Date Requested Status
Kees Cook Approve
Phillip Susi (community) Needs Resubmitting
Marc Deslauriers Needs Fixing
Review via email: mp+53562@code.launchpad.net

Description of the change

02_dont_set_default.patch: SRU patch to make the "remember this
application" check box in the open with dialog box default
to off (LP: #662194)

To post a comment you must log in.
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Could you please update this with the actual patch that was used in Natty? Thanks!

review: Needs Fixing
186. By Phillip Susi

02_dont_set_default.patch: SRU patch to remove the "remember
this application" check box in the open with dialog box
for folders (LP: #662194)

Revision history for this message
Phillip Susi (psusi) wrote :

Updated to the new patch.

review: Needs Resubmitting
Revision history for this message
Kees Cook (kees) wrote :

Looks good, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2010-07-06 12:59:17 +0000
3+++ debian/changelog 2011-03-28 01:24:31 +0000
4@@ -1,3 +1,11 @@
5+nautilus (1:2.30.1-0ubuntu1.2) lucid-proposed; urgency=low
6+
7+ * 02_dont_set_default.patch: SRU patch to remove the "remember
8+ this application" check box in the open with dialog box
9+ for folders (LP: #662194)
10+
11+ -- Phillip Susi <psusi@cfl.rr.com> Sun, 27 Mar 2011 21:20:47 -0400
12+
13 nautilus (1:2.30.1-0ubuntu1.1) lucid-proposed; urgency=low
14
15 * debian/patches/92_git_correct_delay_logic.patch:
16
17=== added file 'debian/patches/02_dont_set_default.patch'
18--- debian/patches/02_dont_set_default.patch 1970-01-01 00:00:00 +0000
19+++ debian/patches/02_dont_set_default.patch 2011-03-28 01:24:31 +0000
20@@ -0,0 +1,29 @@
21+# Upstream: https://bugzilla.gnome.org/show_bug.cgi?id=637264
22+# Ubuntu: https://bugs.launchpad.net/ubuntu/+source/nautilus/+bug/662194
23+# Description: hide checkbox to remember selection when opening folders
24+diff --git a/libnautilus-private/nautilus-open-with-dialog.c b/libnautilus-private/nautilus-open-with-dialog.c
25+index 357e20c..300a92c 100644
26+--- a/libnautilus-private/nautilus-open-with-dialog.c
27++++ b/libnautilus-private/nautilus-open-with-dialog.c
28+@@ -864,7 +864,7 @@ nautilus_open_with_dialog_init (NautilusOpenWithDialog *dialog)
29+
30+ /* Add remember this application checkbox - only visible in open mode */
31+ dialog->details->checkbox = gtk_check_button_new ();
32+- gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (dialog->details->checkbox), TRUE);
33++ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (dialog->details->checkbox), FALSE);
34+ gtk_button_set_use_underline (GTK_BUTTON (dialog->details->checkbox), TRUE);
35+ gtk_widget_show (GTK_WIDGET (dialog->details->checkbox));
36+ gtk_box_pack_start (GTK_BOX (vbox), dialog->details->checkbox, FALSE, FALSE, 0);
37+@@ -1031,6 +1031,12 @@ set_uri_and_type (NautilusOpenWithDialog *dialog,
38+ gtk_label_set_text_with_mnemonic (GTK_LABEL (dialog->details->open_label),
39+ _("_Add"));
40+ gtk_window_set_title (GTK_WINDOW (dialog), _("Add Application"));
41++ } else {
42++ if (g_str_equal (mime_type, "inode/directory")) {
43++ gtk_widget_hide (dialog->details->checkbox);
44++ } else {
45++ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (dialog->details->checkbox), TRUE);
46++ }
47+ }
48+
49+ gtk_label_set_markup (GTK_LABEL (dialog->details->label), label);
50
51=== modified file 'debian/patches/series'
52--- debian/patches/series 2010-07-06 12:59:17 +0000
53+++ debian/patches/series 2011-03-28 01:24:31 +0000
54@@ -1,4 +1,5 @@
55 01_lpi.patch
56+02_dont_set_default.patch
57 03_menu_entry.patch
58 #10_location_titlebar.patch
59 10_load_session.patch

Subscribers

People subscribed via source and target branches

to all changes: