784797 - Excessive left margin in conversation pane

Status: RESOLVED FIXED

(Thunderbird :: Instant Messaging, defect)

Description

[Joe Sabash [:JoeS1]]

Attachment: screenshot

Since 20120822 the conversation pane show an excessive left margin.
bug 768919 ?
It looks fine in win 7 on a wide reso monitor, but pretty ugly on my laptop and oldish winxp systems.

Comment 1

[Richard Marti (:Paenglab)]

Andreas, what do you think about this Bug? 150px may be a little bit wide and gives a lot of white space. We could use 12ch instead of 150px. The advantage would be the width would be adapted depending the font size.

Comment 2

[Florian Quèze [:florian]]

Would a size in % work in this case?

Comment 3

[Richard Marti (:Paenglab)]

Something like nick is 15% of chat width? Wouldn't this make the nick go in overflow earlier on narrow windows and then harder to recognize?

Comment 4

[Florian Quèze [:florian]]

What about a min-width and a max-width in px or ch, and a width in % ?
(mostly thinking out loud at this point; I'm not saying this is what we should do.)

Comment 5

[Andreas Nilsson (:andreasn)]

(In reply to Richard Marti [:Paenglab] from comment #1)
> Andreas, what do you think about this Bug? 150px may be a little bit wide
> and gives a lot of white space. We could use 12ch instead of 150px. The
> advantage would be the width would be adapted depending the font size.

That sounds good to me.

Comment 6

[Richard Marti (:Paenglab)]

Attachment: patch

Patch only changing the width to 12ch. This saves on my system 40px. With this width the nick 'cricccoulson' is fully visible. With 10ch it is already shortend and has an ellipsis at the end.

If this patch is used on TB < 17 then calc() needs to be changed to -moz-calc()

Comment 7

[Andreas Nilsson (:andreasn)]

Comment on attachment 656129 [details] [diff] [review]
patch

ch seems to give issues with bold text

Comment 9

[Richard Marti (:Paenglab)]

Attachment: Patch v2

This patch is now using a width of 15% for the pseudo. To make a gap between pseudo and message I used calc(). For TBs <17 this would need -moz-calc(). To help to identify the pseudo when it's abbreviated, hovering it shows now a tooltip.

Comment 10

[Andreas Nilsson (:andreasn)]

Comment on attachment 659491 [details] [diff] [review]
Patch v2

Looks good to me ui-wise and don't have the issue with boldtext that the previous patch had.
I'm also ok with the labels.

Comment 11

[Florian Quèze [:florian]]

Comment on attachment 659491 [details] [diff] [review]
Patch v2

Should we also take this for Thunderbird 16 and 17?

Comment 12

[Richard Marti (:Paenglab)]

Comment on attachment 659491 [details] [diff] [review]
Patch v2

[Approval Request Comment]
This impoves the display on small screens

Comment 13

[Richard Marti (:Paenglab)]

Attachment: Patch for beta

[Approval Request Comment]
This improves the display on small screens.

The only change to the other patch is -moz-calc() instead of calc() which is supported beginning with Gecko 17.

Comment 14

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/117fc5dee3ee

Comment 15

[Mark Banner (:standard8)]

Checked in:

https://hg.mozilla.org/releases/comm-aurora/rev/696012ce0843
https://hg.mozilla.org/releases/comm-beta/rev/5890b17265c7