Closed Bug 631208 Opened 9 years ago Closed 8 years ago

Bug 553937 regressed bug 121583

Categories

(SeaMonkey :: Themes, defect)

Product:
SeaMonkey
Component:
Themes
Type:
defect
Priority:
Not set

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: neil, Assigned: neil)

References

Details

(Keywords: modern)

Attachments

(1 file)

Proposed patch
564 bytes, patch
iann_bugzilla
: review+
Details | Diff | Splinter Review 
Bug 121583 sets a minheight on the tree's scrollbar to prevent it from affecting layout. But unfortunately that doesn't work if you set a CSS min-height on the scrollbar itself. Switching it back to the slider seems to fix the bug.
Attached patch Proposed patch — — Splinter Review 
Note that I noticed this while browsing CSS Rules in DOM Inspector; without the patch, the position of the vertical splitter will move when the one of the scrollbars appears or disappears. (Do this in a clean profile so you don't have any persisted attributes to confuse matters!)
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #509406 - Flags: review?(iann_bugzilla)
Attachment #509406 - Flags: review?(iann_bugzilla) → review+
Pushed changeset 8b9494bc2d2d to comm-central... a year ago ;-)
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.