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 732882 - Message dialogs - padding and text justification issues
Message dialogs - padding and text justification issues
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: Other
3.13.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
3.14
Depends on:
Blocks:
 
 
Reported: 2014-07-08 08:56 UTC by Allan Day
Modified: 2014-08-03 18:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (2.43 KB, patch)
2014-07-08 09:08 UTC, Allan Day
none Details | Review
alternative patch (1.08 KB, patch)
2014-07-08 09:24 UTC, Allan Day
committed Details | Review
patch comparison screenshot (74.30 KB, image/png)
2014-07-08 09:29 UTC, Allan Day
  Details
patch - left-justified text throughout (1.72 KB, patch)
2014-07-09 10:18 UTC, Allan Day
none Details | Review
comparison with short vs long text (without manual linebreaks) (91.88 KB, image/png)
2014-07-11 15:15 UTC, Matthias Clasen
  Details

Description Allan Day 2014-07-08 08:56:13 UTC
Message dialogs don't look great right now. There is too much padding above the content, and the mix of centered and left-justified text can often look odd and unbalanced.
Comment 1 Allan Day 2014-07-08 09:08:04 UTC
Created attachment 280125 [details] [review]
patch

This removes the extra vertical space and uses centered padding throughout. It looks a lot better, in my opinion.
Comment 2 Lapo Calamandrei 2014-07-08 09:12:30 UTC
As I said on irc, I not a great fan of centered text, since it makes reading harder, frankly speaking I prefer the current way (centered title + left aligned text) modulo vertical padding.
Comment 3 Allan Day 2014-07-08 09:24:02 UTC
Created attachment 280126 [details] [review]
alternative patch

We should remove the unnecessary padding, whatever happens.
Comment 4 Allan Day 2014-07-08 09:29:47 UTC
Created attachment 280127 [details]
patch comparison screenshot

Left-aligned body with a centred heading works if you have a long block of body text, but with a short body text (as is typical for message dialogs), there's no identifiable alignment point on the right. This leaves the dialog looking unbalanced - you have a line on the left, but none on the right. This leaves the heading looking unanchored on one side.
Comment 5 Jakub Steiner 2014-07-08 11:55:28 UTC
Definitely agree that mixing a centered header and left aligned block makes no sense, however I lean towards left aligning both to having multiline centered block. I am fine with pushing that through though.
Comment 6 Allan Day 2014-07-09 10:18:01 UTC
Created attachment 280219 [details] [review]
patch - left-justified text throughout

(In reply to comment #5)
> Definitely agree that mixing a centered header and left aligned block makes no
> sense, however I lean towards left aligning both to having multiline centered
> block. I am fine with pushing that through though.

I'd be happy to try left-aligned.

I had some issues with this patch - the heading and body text of the message dialog shown by the inspector aren't aligned. Not sure if that's an issue with that specific dialog or whether I'm getting it wrong.
Comment 7 Matthias Clasen 2014-07-11 15:09:21 UTC
I should probably point out that the example dialog in widget-factory looks the way it does because it has an explicit linebreak (I was trying to reproduce the HIG example). If you take that out, the secondary text comes out as a single line.
Comment 8 Matthias Clasen 2014-07-11 15:15:08 UTC
Created attachment 280518 [details]
comparison with short vs long text (without manual linebreaks)
Comment 9 Matthias Clasen 2014-08-03 18:39:32 UTC
Lets just call it good enough for now