Merge lp:~larsu/nautilus/set-max-width-for-properties-window into lp:~ubuntu-desktop/nautilus/ubuntu

Proposed by Lars Karlitski
Status: Merged
Merged at revision: 439
Proposed branch: lp:~larsu/nautilus/set-max-width-for-properties-window
Merge into: lp:~ubuntu-desktop/nautilus/ubuntu
Diff against target: 55 lines (+35/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/git_properties_window_set_max_width_for_value_labels.patch (+27/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~larsu/nautilus/set-max-width-for-properties-window
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Review via email: mp+237301@code.launchpad.net

Commit message

* debian/patches/properties_window_set_max_width_for_value_labels.patch:
  - avoid very wide properties window (GNOME: #732117)

Description of the change

* debian/patches/properties_window_set_max_width_for_value_labels.patch:
  - avoid very wide properties window (GNOME: #732117)

To post a comment you must log in.
Revision history for this message
Lars Karlitski (larsu) wrote :

Just pushed this to upstream master. Should I rename the patch to git_*?

Revision history for this message
Sebastien Bacher (seb128) wrote :

renaming to git_ would be nicer, looks good to commit otherwise

review: Approve
440. By Lars Karlitski

Rename properties_window_set_max_width_for_value_labels

It was accespted upstream.

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 2014-09-30 16:59:37 +0000
3+++ debian/changelog 2014-10-08 07:59:36 +0000
4@@ -1,3 +1,10 @@
5+nautilus (1:3.10.1-0ubuntu15) UNRELEASED; urgency=medium
6+
7+ * debian/patches/properties_window_set_max_width_for_value_labels.patch:
8+ - avoid very wide properties window (GNOME: #732117)
9+
10+ -- Lars Uebernickel <lars.uebernickel@ubuntu.com> Mon, 06 Oct 2014 17:23:30 +0200
11+
12 nautilus (1:3.10.1-0ubuntu14) utopic; urgency=medium
13
14 * No-change rebuild for gnome-desktop 3.12
15
16=== added file 'debian/patches/git_properties_window_set_max_width_for_value_labels.patch'
17--- debian/patches/git_properties_window_set_max_width_for_value_labels.patch 1970-01-01 00:00:00 +0000
18+++ debian/patches/git_properties_window_set_max_width_for_value_labels.patch 2014-10-08 07:59:36 +0000
19@@ -0,0 +1,27 @@
20+From 0618af5a154cff26c7a04b303abcbeec2a8ca10d Mon Sep 17 00:00:00 2001
21+From: Lars Uebernickel <lars.uebernickel@canonical.com>
22+Date: Mon, 6 Oct 2014 13:07:51 +0200
23+Subject: [PATCH] properties-window: set max-width for value labels
24+
25+Not setting it can lead to very wide windows, because gtk doesn't
26+enforce a maximum window size anymore.
27+
28+https://bugzilla.gnome.org/show_bug.cgi?id=732117
29+---
30+ src/nautilus-properties-window.c | 1 +
31+ 1 file changed, 1 insertion(+)
32+
33+diff --git a/src/nautilus-properties-window.c b/src/nautilus-properties-window.c
34+index 3b15dd9..631c456 100644
35+--- a/src/nautilus-properties-window.c
36++++ b/src/nautilus-properties-window.c
37+@@ -1276,6 +1276,7 @@ attach_label (GtkGrid *grid,
38+
39+ if (ellipsize_text) {
40+ gtk_widget_set_hexpand (label_field, TRUE);
41++ gtk_label_set_max_width_chars (GTK_LABEL (label_field), 24);
42+ }
43+
44+ if (sibling != NULL) {
45+--
46+2.1.0
47
48=== modified file 'debian/patches/series'
49--- debian/patches/series 2014-09-29 16:57:17 +0000
50+++ debian/patches/series 2014-10-08 07:59:36 +0000
51@@ -42,3 +42,4 @@
52 0001-Don-t-try-to-open-a-selection-if-we-don-t-have-one.patch
53 git_also_tracker-1.0.patch
54 ubuntu_tracker_only_on_GNOME.patch
55+git_properties_window_set_max_width_for_value_labels.patch

Subscribers

People subscribed via source and target branches

to all changes: