762129 - Wrong behavior of the Participants/Previous conversations splitter when moved all the way to the top/bottom

Status: RESOLVED FIXED

(Thunderbird :: Instant Messaging, defect)

Description

[Florian Quèze [:florian]]

Attachment: Patch

Currently moving the splitter between the list of participants and the list of previous conversations all the way to the bottom makes a vertical scrollbar appear, and moving it all the way to the top moves the "Participants" header into the toolbar displaying the conversation title and topic.

This can easily be fixed by adding another vbox to limit how much the splitter can move the lists to the top, and adding overflow:hidden to it to avoid the scrollbar.

Comment 1

[Florian Quèze [:florian]]

Attachment: Same patch with -w for easy review

Comment 2

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 630617 [details] [diff] [review]
Patch

Review of attachment 630617 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks Florian.

Comment 3

[Florian Quèze [:florian]]

https://hg.mozilla.org/comm-central/rev/f99dd90965a1

Comment 4

[Florian Quèze [:florian]]

Comment on attachment 630617 [details] [diff] [review]
Patch

[Approval Request Comment]
Simple bugfix for the Chat feature.

Comment 5

[Florian Quèze [:florian]]

https://hg.mozilla.org/releases/comm-aurora/rev/4517e86cbcf6