968814 - Frames added in Toolbar have scrollbars if they have any content in them

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect, P3)

Description

[Matteo Ferretti [:zer0] [:matteo]]

From an add-on like this: https://gist.github.com/wbamberg/8725047

Comment 2

[Tomislav Jovanovic :zombie]

from bug 972935, it looks like this is a problem across platforms..

Comment 3

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Attachment: Fix

Comment 4

[[github robot]]

Commit pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/42aa9919e109921be63c311af4c0eef2f0b5562f
Merge pull request #1407 from Gozala/bug/toolbar-scroll

fix Bug 968814 - Fix undesired scrollbars in toolbars. r=@ZER0

Comment 5

[Dave Townsend [:mossop]]

Attachment: patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): New SDK toolbars
User impact if declined: On some older OSX platforms the toolbar will show a small scrollbar
Testing completed (on m-c, etc.): On Nightly for over a week, little add-on code uses this though so probably little real-world testing.
Risk to taking this patch (and alternatives if risky): The change is just some styling corrections, very low risk.
String or IDL/UUID changes made by this patch: None

Comment 6

[Dave Townsend [:mossop]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/6f3804e364bd