wrapped nick causes extra height on secondary lines

RESOLVED FIXED in Thunderbird 19.0


6 years ago
6 years ago


(Reporter: mark.yen, Assigned: Mook)


Thunderbird 19.0

Firefox Tracking Flags

(Not tracked)



(2 attachments, 1 obsolete attachment)



6 years ago
Created attachment 670518 [details] [diff] [review]
use display:none

When a user's nick wraps, that message line becomes double-height (to display the nick).  However, when the nick is hidden (because it's a subsequent message from the same user), that extra height is still used.

The attached patch switches it from visibility:hidden to display:none to avoid this.
Attachment #670518 - Flags: review?(nisses.mail)

Comment 1

6 years ago
Created attachment 670519 [details]
screenshot of the problem
Attachment #670518 - Flags: review?(nisses.mail) → review?(richard.marti)
Comment on attachment 670518 [details] [diff] [review]
use display:none

This looks good and does for what it is made. But it would be better to stop wrapping the nick. When a wrapped nick sends only a short sentence like 'ping' or 'okay', it still uses two lines for the nick.

A .pseudo (white-space: nowrap;) would stop the wrapping and abbreviate the wrappable nicks like normal long nicks.

Please could you do a patch like this? You can also include your first patch in the new patch.
Attachment #670518 - Flags: review?(richard.marti) → review-

Comment 3

6 years ago
Created attachment 673109 [details] [diff] [review]
set white-space: nowrap on .pseudo

Actually, with .pseudo { white-space: nowrap; }, I don't even need the display: none change anymore; everything has at least one line anyway, so whether it's visibility: hidden or display: none doesn't matter very much.

Of course, this means that if the _contents_ first line does wrap, the name still wouldn't.  Ah well, I don't think there's a sensible way of fixing that.
Assignee: nobody → mook.moz+mozbz
Attachment #670518 - Attachment is obsolete: true
Attachment #673109 - Flags: review?(richard.marti)
Comment on attachment 673109 [details] [diff] [review]
set white-space: nowrap on .pseudo

This looks good. r+

Thank you for the patch
Attachment #673109 - Flags: review?(richard.marti) → review+
setting checkin-needed will make this patch land in the tree.
Keywords: checkin-needed

Comment 6

6 years ago

... I hope I haven't screwed up pushing things!
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 19.0
You need to log in before you can comment on or make changes to this bug.