Message header not updating its height properly when handling long subject line
Categories
(Thunderbird :: General, defect, P1)
Tracking
(Not tracked)
People
(Reporter: aleca, Assigned: khushil324)
References
Details
(Keywords: regression, regressionwindow-wanted)
Attachments
(4 files)
Screenshot from 2020-07-13 14-18-46.png
The header height doesn't update properly when selecting a message with a long subject that causes multiple lines.
As you can see from the screenshot, the To line is not visible but present and cut off that area.
This seems to happen only if there's already a previously selected and visualized message with a header height attribute set.
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 1•4 years ago
|
||
Are you able to reproduce this issue on the trunk or is it a beta specific issue? I am not able to reproduce it on the Trunk.
We have added this function during de-grid task: https://searchfox.org/comm-central/search?q=OnResizeExpandedHeaderView&path=
|
Reporter | |
Comment 2•4 years ago
|
||
Yes, it's on trunk.
I was able to reproduce it consistently. The issue happens only if a previously selected message header has only 1 single line, with a very short message.
Here's a screencap.
|
|
Assignee | |
Comment 3•4 years ago
|
||
I am not seeing this on macOS. I guess you are using Linux. When I change the message, the header portion's height is getting changed every time. If you have a macOS, can you check?
|
||
Comment 4•4 years ago
|
||
With TB 78 I see this too on Windows. With Daily I don't see this on Windows. Also tried on Linux with Daily and the header expands and shows all.
|
|
Assignee | |
Comment 5•4 years ago
|
||
|
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 6•4 years ago
|
||
Richard, can you check this with TB 78 on Windows? Also, if you have a TB 78 build on Linux.
|
|
||
Comment 7•4 years ago
|
||
|
|
Assignee | |
Comment 8•4 years ago
|
||
Can you debug this? We just need to call OnResizeExpandedHeaderView or tweak the function to make it work. Right now, OnResizeExpandedHeaderView gets called when we resize the messenger.xhtml or messageWindow.xhtml.
|
|
||
Comment 9•4 years ago
|
||
I can't debug. Maybe Alessandro can.
|
|
Reporter | |
Comment 10•4 years ago
|
||
Tricky beast indeed.
I'll see if I have some capacity to debug this later today.
Thanks Khushil for taking on this annoying issue, we'll get to the bottom of this 👍️
|
|
Assignee | |
Comment 11•4 years ago
|
||
If you are busy then I will try on my Linux machine with Beta build. It seems that we need to call a function in the right place.
|
||
Updated•4 years ago
|
|
|
Assignee | |
Comment 12•4 years ago
|
||
Wayne, are you able to reproduce this in any particular way?
I have tried but I am not able to reproduce this issue on MacOS 78 and daily, Linux Ubuntu 78 and daily. This is probably Windows 78 issue.
|
|
||
Comment 13•4 years ago
|
||
no, can't say I have seen this.
|
||
Comment 15•4 years ago
|
||
(In reply to Wayne Mery (:wsmwk) from comment #14)
TCW can you reproduce?
Well, I'm testing with 84.0b here at home and I'm not seeing it there.
|
|
||
Comment 16•4 years ago
|
||
(In reply to Wayne Mery (:wsmwk) from comment #14)
TCW can you reproduce?
And I also tested with TB Portable 78.5.0. I'm seeing the To: field just fine with headers set to Normal.
|
|
Reporter | |
Comment 17•4 years ago
|
||
I'm not able to reproduce this issue anymore.
Description
•