Regression of min-/max-height on -moz-box containers

RESOLVED FIXED in Firefox 17

Status

()

Core
XUL
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Mats Palmgren (vacation - back in August), Assigned: Mats Palmgren (vacation - back in August))

Tracking

({regression})

unspecified
mozilla18
regression
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox16-, firefox17+ fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
Created attachment 659591 [details] [diff] [review]
fix + reftest

https://tbpl.mozilla.org/?tree=Try&rev=23519df003cd
Attachment #659591 - Flags: review?(bzbarsky)
tracking-firefox16: --- → ?
tracking-firefox17: --- → ?
tracking-firefox16: ? → +
tracking-firefox17: ? → +
Comment on attachment 659591 [details] [diff] [review]
fix + reftest

r=me.  Sorry I missed this in bug 785754.  :(
Attachment #659591 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/c14834c3ce2d
Flags: in-testsuite+
Target Milestone: --- → mozilla18
Comment on attachment 659591 [details] [diff] [review]
fix + reftest

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 776265
User impact if declined: minor layout error when using CSS min-/max-height on display:-moz-box elements (I'd expect that's used in XUL for the most part)
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): low risk
String or UUID changes made by this patch:  none
Attachment #659591 - Flags: approval-mozilla-beta?
Attachment #659591 - Flags: approval-mozilla-aurora?

Comment 5

5 years ago
https://hg.mozilla.org/mozilla-central/rev/c14834c3ce2d
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Comment on attachment 659591 [details] [diff] [review]
fix + reftest

We're backing out bug 716875 for Fx16, so this isn't needed there.
Attachment #659591 - Flags: approval-mozilla-beta?
Attachment #659591 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Taking off tracking 16 here then since the backout will be tracked in bug 716875.
tracking-firefox16: + → -
https://hg.mozilla.org/releases/mozilla-aurora/rev/3bb78dd08489
status-firefox17: --- → fixed
You need to log in before you can comment on or make changes to this bug.