After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 747148 - "Failed To Update" message box may be too large to fit into any screen
"Failed To Update" message box may be too large to fit into any screen
Status: RESOLVED FIXED
Product: gnome-software
Classification: Applications
Component: General
3.16.x
Other Linux
: Normal normal
: ---
Assigned To: GNOME Software maintainer(s)
GNOME Software maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2015-03-31 23:58 UTC by Rafal Luzynski
Modified: 2015-04-14 09:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Too large "Failed To Update" message box (572.13 KB, image/png)
2015-03-31 23:58 UTC, Rafal Luzynski
  Details
Proposed solution (2.31 KB, patch)
2015-04-07 18:12 UTC, Rafal Luzynski
none Details | Review
New look of the "Failed To Update" message dialog (284.44 KB, image/png)
2015-04-07 18:17 UTC, Rafal Luzynski
  Details

Description Rafal Luzynski 2015-03-31 23:58:37 UTC
Created attachment 300720 [details]
Too large "Failed To Update" message box

See the screenshot attached. The reason why it does not fit is that the error text is displayed with the standard GtkMessageDialog which does not work good with very long texts. We need a simple custom dialog box which looks almost the same as the GtkMessageDialog but features a GtkScrolledWindow inside.

Also I suggest to view other occurrences of GtkMessageDialog and check if we are sure that the length of the message text is limited.

If you wonder how I have achieved this bug: I have launched the offline update with many pending updates and no free disk space.
Comment 1 Rafal Luzynski 2015-04-07 18:12:40 UTC
Created attachment 301087 [details] [review]
Proposed solution

This is not sophisticated but simple and working solution to this problem. This patch inserts a GtkScrolledWindow containing the message text into the message dialog. The height of the scrolled window is set to the height of the text label inside but no more than 300 pixels. The vertical scrollbar appears only if the label height is larger than the scrolled window so for smaller texts there should be no visual difference.
Comment 2 Rafal Luzynski 2015-04-07 18:17:53 UTC
Created attachment 301088 [details]
New look of the "Failed To Update" message dialog
Comment 3 Richard Hughes 2015-04-14 09:05:30 UTC
Pushed to master, thanks!