Closed
Bug 1423383
(enable-column-span-nightly)
Opened 7 years ago
Closed 5 years ago
Enable pref for column-span (layout.css.column-span.enabled) by default for Nightly only.
Categories
(Core :: Layout: Columns, enhancement, P3)
Core
Layout: Columns
Tracking
()
RESOLVED
FIXED
mozilla71
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: neerja, Assigned: TYLin)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(1 file, 1 obsolete file)
No description provided.
Reporter | ||
Updated•6 years ago
|
Summary: Enable pref for column-span (layout.css.column-span.enabled) by default. → Enable pref for column-span (layout.css.column-span.enabled) by default for Nightly only.
Updated•6 years ago
|
Priority: -- → P1
Updated•6 years ago
|
Priority: P1 → P3
Assignee | ||
Updated•6 years ago
|
Component: Layout → Layout: Columns
Comment 1•6 years ago
|
||
Once this landed in Nightly, it should be mentioned on https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Experimental_features#column-span. Sebastian
Keywords: dev-doc-needed
Assignee | ||
Updated•6 years ago
|
Depends on: fuzzing-column-span
Assignee | ||
Updated•6 years ago
|
Depends on: ship-column-span
Assignee | ||
Updated•6 years ago
|
Blocks: ship-column-span
No longer depends on: ship-column-span
Comment 2•5 years ago
|
||
Neerja hasn't been active in bugzilla since February 2018.
Assignee: neerjapancholi → nobody
Depends on: 1565665
Assignee | ||
Comment 3•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → aethanyc
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•5 years ago
|
||
properties-db.js is patched via ./mach devtools-css-db
.
Updated•5 years ago
|
Attachment #9084449 -
Attachment description: Bug 1423383 - Enable layout.css.column-span.enabled on Nightly. → Bug 1423383 Part 2 - Enable layout.css.column-span.enabled on Nightly.
Updated•5 years ago
|
Attachment #9093377 -
Attachment description: Bug 1423383 Part 1 - Update devtools' properties-db.js, and update test expectations. → Bug 1423383 Part 1 - Update devtools' properties-db.js for column-span, and update test expectations.
Comment 5•5 years ago
|
||
Are these patches out-of-order?
./mach devtools-css-db
makes no changes on a clean up-to-date mozilla-central build, so the commit message in part 1 seems possibly-wrong ("properties-db.js is patched via ./mach devtools-css-db.")
I'm guessing that patch was generated by first building with the pref-flip, and then running the devtools mach command. Which makes me think the patches should be in the opposite order...
Flags: needinfo?(aethanyc)
Comment 6•5 years ago
|
||
(Or they should just be folded into a single patch perhaps, since neither patch will pass tests on its own I think.)
Assignee | ||
Comment 7•5 years ago
|
||
Oops, you're right. ./mach devtools-css-db
makes no changes if the pref is not flipped first ... I'll fold the two patches into one.
Flags: needinfo?(aethanyc)
Updated•5 years ago
|
Attachment #9084449 -
Attachment description: Bug 1423383 Part 2 - Enable layout.css.column-span.enabled on Nightly. → Bug 1423383 - Enable layout.css.column-span.enabled on Nightly.
Updated•5 years ago
|
Attachment #9093377 -
Attachment is obsolete: true
Pushed by aethanyc@gmail.com: https://hg.mozilla.org/integration/autoland/rev/db7389169402 Enable layout.css.column-span.enabled on Nightly. r=dholbert
Comment 9•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox71:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Updated•5 years ago
|
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•