Closed Bug 1295788 Opened 8 years ago Closed 8 years ago

Remove layout.css.display-contents.enabled pref

Categories

(Core :: CSS Parsing and Computation, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox51 --- wontfix
firefox53 --- fixed

People

(Reporter: MatsPalmgren_bugz, Assigned: chenpighead)

References

Details

(Keywords: dev-doc-complete, Whiteboard: [lang=c++][good second bug] )

Attachments

(1 file)

display:contents was enabled by default in v37 (bug 1105369), so I think
we can remove the layout.css.display-contents.enabled pref now.
Depends on: 1105369
Priority: -- → P3
Assignee: nobody → jeremychen
Status: NEW → ASSIGNED
Comment on attachment 8811182 [details]
Bug 1295788 - Remove layout.css.display-contents.enabled pref.

https://reviewboard.mozilla.org/r/93384/#review93456

LGTM, thanks.
Attachment #8811182 - Flags: review?(mats) → review+
Pushed by jichen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b5ee658f3937
Remove layout.css.display-contents.enabled pref.
https://hg.mozilla.org/mozilla-central/rev/b5ee658f3937
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
We've built 51 RC. Mark 51 won't fix.
I've updated the browser compat information on the display page to mention this (see footnote 2)

https://developer.mozilla.org/en-US/docs/Web/CSS/display#Browser_compatibility

I've also added a note to the Fx53 release notes:

https://developer.mozilla.org/en-US/Firefox/Releases/53#CSS

Let me know if you think anything else needs updating.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: