diff --git a/SOURCES/zenity-3.28.1-msgdialog-label.patch b/SOURCES/zenity-3.28.1-msgdialog-label.patch new file mode 100644 index 0000000..72ba895 --- /dev/null +++ b/SOURCES/zenity-3.28.1-msgdialog-label.patch @@ -0,0 +1,92 @@ +From 4f1e06a69e42ce8dc56f2101b98e1a1021fd324d Mon Sep 17 00:00:00 2001 +From: David King +Date: Mon, 25 Apr 2022 13:42:41 +0100 +Subject: [PATCH 1/2] msg: Fix maximum width to 60 characters + +Commit 3b64d05e8a1aae1581d2bec7288d80ac4699e1b1 mentioned that labels in +message dialogs were set to a maximum width of 60 characters, with +inspiration from GtkMessageDialog, but in fact a typo meant that they +were set to both a width and maximum width of just 10 characters. This +lead to text layout problems, which can be fixed by using the intended +maximum of 60 characters. + +https://bugzilla.redhat.com/show_bug.cgi?id=2053530 +--- + src/msg.c | 10 ++++------ + 1 file changed, 4 insertions(+), 6 deletions(-) + +diff --git a/src/msg.c b/src/msg.c +index ca14d86..9265152 100644 +--- a/src/msg.c ++++ b/src/msg.c +@@ -202,12 +202,10 @@ zenity_msg (ZenityData *data, ZenityMsgData *msg_data) { + if (data->width > -1) + gtk_widget_set_size_request (GTK_WIDGET (text), data->width, -1); + else if (!msg_data->ellipsize && !msg_data->no_wrap) { +- // the magic number 60 is picked from gtk+/gtk/ui/gtkmessagedialog.ui +- // however, 60 would increase the distance between the icon and the +- // text, +- // decreasing to 10 fix it. +- gtk_label_set_width_chars (text, 10); +- gtk_label_set_max_width_chars (text, 10); ++ /* The magic number 60 is taken from gtk+/gtk/ui/gtkmessagedialog.ui ++ with 10 as a minimum width. */ ++ gtk_label_set_width_chars (GTK_LABEL (text), 10); ++ gtk_label_set_max_width_chars (GTK_LABEL (text), 60); + } + + if (data->modal) +-- +2.36.1 + + +From 28f865111bb890a7af06a7767f16f2502aa2f213 Mon Sep 17 00:00:00 2001 +From: David King +Date: Tue, 26 Apr 2022 08:18:18 +0100 +Subject: [PATCH 2/2] zenity.ui: make message dialogs non-resizable + +Use the approach of GtkMessageDialog, to avoid problems with narrow +labels when the dialog text is long. +--- + src/zenity.ui | 4 ++++ + 1 file changed, 4 insertions(+) + +diff --git a/src/zenity.ui b/src/zenity.ui +index 1359442..546a622 100644 +--- a/src/zenity.ui ++++ b/src/zenity.ui +@@ -465,6 +465,7 @@ + Error + center + dialog ++ false + + + +@@ -681,6 +682,7 @@ + Information + center + dialog ++ false + + + +@@ -882,6 +884,7 @@ + Question + center + dialog ++ false + + + +@@ -1064,6 +1067,7 @@ + Warning + center + dialog ++ false + + + +-- +2.36.1 + diff --git a/SPECS/zenity.spec b/SPECS/zenity.spec index d0ba625..c5f13ee 100644 --- a/SPECS/zenity.spec +++ b/SPECS/zenity.spec @@ -1,11 +1,13 @@ Name: zenity Version: 3.28.1 -Release: 1%{?dist} +Release: 2%{?dist} Summary: Display dialog boxes from shell scripts License: LGPLv2+ URL: https://wiki.gnome.org/Projects/Zenity Source: https://download.gnome.org/sources/%{name}/3.28/%{name}-%{version}.tar.xz +# https://bugzilla.redhat.com/show_bug.cgi?id=2053530 +Patch0: zenity-3.28.1-msgdialog-label.patch BuildRequires: pkgconfig(gtk+-3.0) >= 3.0.0 BuildRequires: pkgconfig(libnotify) >= 0.6.1 @@ -20,7 +22,7 @@ shell scripts. It is similar to gdialog, but is intended to be saner. It comes from the same family as dialog, Xdialog, and cdialog. %prep -%setup -q +%autosetup -p1 %build @@ -46,6 +48,9 @@ rm $RPM_BUILD_ROOT%{_bindir}/gdialog %changelog +* Wed May 11 2022 David King - 3.28.1-2 +- Fix message dialog label layout (#2053530) + * Tue Apr 10 2018 Kalev Lember - 3.28.1-1 - Update to 3.28.1