Merge lp:~chrisccoulson/nautilus/bug362771 into lp:~ubuntu-desktop/nautilus/ubuntu

Proposed by Chris Coulson
Status: Merged
Merged at revision: not available
Proposed branch: lp:~chrisccoulson/nautilus/bug362771
Merge into: lp:~ubuntu-desktop/nautilus/ubuntu
Diff against target: None lines
To merge this branch: bzr merge lp:~chrisccoulson/nautilus/bug362771
Reviewer Review Type Date Requested Status
Sebastien Bacher Pending
Review via email: mp+5728@code.launchpad.net
To post a comment you must log in.

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 2009-04-16 13:43:03 +0000
3+++ debian/changelog 2009-04-20 22:52:17 +0000
4@@ -1,9 +1,15 @@
5 nautilus (1:2.26.2-0ubuntu2) UNRELEASED; urgency=low
6
7+ [ Michael Vogt ]
8 * debian/patches/82_gnome-app-install.patch:
9 - re-enable the mime search functionality (LP: #362307)
10
11- -- Michael Vogt <michael.vogt@ubuntu.com> Thu, 16 Apr 2009 14:51:09 +0200
12+ [ Chris Coulson ]
13+ * debian/patches/18_svn_r15184_fix_icon_view_zoom.patch:
14+ - Patch from SVN to take zoom in to account when updating
15+ the scroll width (LP: #362771).
16+
17+ -- Chris Coulson <chrisccoulson@googlemail.com> Mon, 20 Apr 2009 23:34:59 +0100
18
19 nautilus (1:2.26.2-0ubuntu1) jaunty; urgency=low
20
21
22=== added file 'debian/patches/18_svn_r15184_fix_icon_view_zoom.patch'
23--- debian/patches/18_svn_r15184_fix_icon_view_zoom.patch 1970-01-01 00:00:00 +0000
24+++ debian/patches/18_svn_r15184_fix_icon_view_zoom.patch 2009-04-20 22:52:17 +0000
25@@ -0,0 +1,30 @@
26+# Excessive white-space on right-hand side in icon view
27+# When using at allocation->width, take zoom into account
28+# Launchpad: https://bugs.launchpad.net/ubuntu/+source/nautilus/+bug/362771
29+# Bugzilla: http://bugzilla.gnome.org/show_bug.cgi?id=579086
30+
31+Index: nautilus-2.26.2/libnautilus-private/nautilus-icon-container.c
32+===================================================================
33+--- nautilus-2.26.2.orig/libnautilus-private/nautilus-icon-container.c 2009-04-20 23:31:26.000000000 +0100
34++++ nautilus-2.26.2/libnautilus-private/nautilus-icon-container.c 2009-04-20 23:31:36.000000000 +0100
35+@@ -1033,9 +1033,9 @@
36+ GtkAllocation *allocation;
37+ gboolean reset_scroll_region;
38+
39+- if (nautilus_icon_container_get_is_fixed_size (container)) {
40+- pixels_per_unit = EEL_CANVAS (container)->pixels_per_unit;
41++ pixels_per_unit = EEL_CANVAS (container)->pixels_per_unit;
42+
43++ if (nautilus_icon_container_get_is_fixed_size (container)) {
44+ /* Set the scroll region to the size of the container allocation */
45+ allocation = &GTK_WIDGET (container)->allocation;
46+ eel_canvas_set_scroll_region
47+@@ -1090,7 +1090,7 @@
48+ if (nautilus_icon_container_is_auto_layout (container)) {
49+ allocation = &GTK_WIDGET (container)->allocation;
50+ x1 = MIN (x1, 0);
51+- x2 = MAX (x2, allocation->width);
52++ x2 = MAX (x2, allocation->width / pixels_per_unit);
53+ y1 = 0;
54+ } else {
55+ /* Otherwise we add the padding that is at the start of the
56
57=== modified file 'debian/patches/series'
58--- debian/patches/series 2009-04-16 13:43:03 +0000
59+++ debian/patches/series 2009-04-20 22:52:17 +0000
60@@ -12,6 +12,7 @@
61 15_translations_list_update.patch
62 16_workaround_autorestart_issue.patch
63 17_no_autorun_launcher.patch
64+18_svn_r15184_fix_icon_view_zoom.patch
65 80_suppress_umount_in_ltsp.patch
66 82_gnome-app-install.patch
67 95_no-initial-fade.patch

Subscribers

People subscribed via source and target branches

to all changes: