Closed
Bug 783409
Opened 12 years ago
Closed 12 years ago
Turn on CSS flexbox in builds by default (by enabling pref, build flag, etc)
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla20
People
(Reporter: dholbert, Assigned: dholbert)
References
Details
(Whiteboard: [pref currently disabled in release builds -- see bug 841876])
Attachments
(1 file, 3 obsolete files)
(deleted),
patch
|
dbaron
:
review+
|
Details | Diff | Splinter Review |
Bug 666041 is on the main CSS3 flexbox implementation, but there may be a bit of followup work to do in helper-bugs that will land separately.
I'm filing this bug to track the work that we need to do before we're ready to enable flexbox in nightly builds.
Assignee | ||
Comment 1•12 years ago
|
||
Here's the patch to toggle flexbox on in builds (formerly posted as "patch 99" on bug 666041)
Assignee | ||
Comment 2•12 years ago
|
||
NOTE: this bug is really a tracker for when flexbox is ready to be enabled in builds & exposed to the web by default.
If it turns out we can easily pref properties off (soon w/ paris DOM bindings -- maybe it's already possible?), then we can enable MOZ_FLEXBOX in builds ASAP, but just pref off its properties (so that developers can toggle the pref to try it out while pieces are still landing). Then, we can morph this bug into preffing it on.
Summary: Enable MOZ_FLEXBOX in builds → Enable MOZ_FLEXBOX in builds (preffed on, if possible/applicable)
Assignee | ||
Updated•12 years ago
|
Summary: Enable MOZ_FLEXBOX in builds (preffed on, if possible/applicable) → Enable MOZ_FLEXBOX in builds (and pref it on, if applicable)
Assignee | ||
Comment 3•12 years ago
|
||
Updated patch to also uncomment the flexbox-reftest line in layout/reftests/reftest.list
Attachment #652616 -
Attachment is obsolete: true
Assignee | ||
Comment 4•12 years ago
|
||
I'm clarifying this bug's summary to be track *fully enabling* flexbox by default.
(I'm going to spin off a separate bug on enabling the build flag, so that the flexbox code will compile but will still be preffed off per bug 796212.)
Depends on: 796212
Summary: Enable MOZ_FLEXBOX in builds (and pref it on, if applicable) → Turn on CSS flexbox in builds by default (by enabling pref, build flag, etc)
Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 666341 [details] [diff] [review]
patch v2 to enable flexbox in builds
(In reply to Daniel Holbert [:dholbert] from comment #4)
> (I'm going to spin off a separate bug on enabling the build flag
Spun off bug 797022 for that. Obsoleting the patch on this bug here -- it's being replaced by a few patches over on that new bug.
Attachment #666341 -
Attachment is obsolete: true
Assignee | ||
Comment 6•12 years ago
|
||
For reference, here's what the patch for this bug would look like now -- just a pref tweak.
Assignee | ||
Comment 7•12 years ago
|
||
I think we should turn this on, at least in nightly builds. (Though we may decide to turn it off again on the Beta and possibly Aurora channels, when this patch makes it to those, depending on how spec-compliant/bug-free we are at that point.)
Patch attached to enable this and update our automated tests' expectations about the pref being [dis|en]abled.
Try run: https://tbpl.mozilla.org/?tree=Try&rev=f8133fcad6ca
Attachment #667393 -
Attachment is obsolete: true
Assignee | ||
Updated•12 years ago
|
Attachment #689451 -
Flags: review?(dbaron)
Comment 8•12 years ago
|
||
Comment on attachment 689451 [details] [diff] [review]
patch v4: enable flexbox pref, and tweak reftests/mochitests accordingly
r=dbaron
Attachment #689451 -
Flags: review?(dbaron) → review+
Assignee | ||
Comment 9•12 years ago
|
||
Assignee | ||
Comment 10•12 years ago
|
||
oops, that had the wrong bug number. (inverted two digits)
Backed out:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4bf567bd1a3c
and re-landed w/ that fixed:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9bf579519470
Comment 11•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
Assignee | ||
Updated•12 years ago
|
Whiteboard: [pref currently disabled in release builds -- see bug 841876]
You need to log in
before you can comment on or make changes to this bug.
Description
•