Closed Bug 762129 Opened 8 years ago Closed 7 years ago

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

Categories

(Thunderbird :: Instant Messaging, defect)

defect
Not set

Tracking

(thunderbird15 fixed)

RESOLVED FIXED
Thunderbird 16.0
Tracking Status
thunderbird15 --- fixed

People

(Reporter: florian, Assigned: florian)

Details

Attachments

(2 files)

Attached patch PatchSplinter Review
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.
Attachment #630617 - Flags: review?(mconley)
Assignee: nobody → florian
Status: NEW → ASSIGNED
Comment on attachment 630617 [details] [diff] [review]
Patch

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

Looks good, thanks Florian.
Attachment #630617 - Flags: review?(mconley) → review+
https://hg.mozilla.org/comm-central/rev/f99dd90965a1
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 16.0
Comment on attachment 630617 [details] [diff] [review]
Patch

[Approval Request Comment]
Simple bugfix for the Chat feature.
Attachment #630617 - Flags: approval-comm-aurora?
Attachment #630617 - Flags: approval-comm-aurora? → approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.