Closed Bug 1307401 Opened 8 years ago Closed 6 years ago

Add `shape-margin` support to the style system

Categories

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

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1265342

People

(Reporter: kuoe0.tw, Assigned: bradwerth)

References

(
URL
)

Details

I'd like to implement `shape-margin` property in style system.
Blocks: 1265342
Summary: Add `shape-maring` support to the style system → Add `shape-margin` support to the style system
Could you please clarify how this differs from bug 1265342?

Sebastian
Flags: needinfo?(kuoe0)
In this bug, I'll only add the CSS property into style system. The rendering part I think we should wait `shape-outside` finished.
Flags: needinfo?(kuoe0)
Ah, I see. Then please don't forget to put the property behind the layout.css.shape-outside.enabled preference, so it's not exposed to web content by default.

Sebastian
Ok, no problem!
Priority: -- → P3
Adding this CSS property looks so trivial that I don't think it's worth being done in a separate bug, especially given you do not plan to finish the layout part soonish.

Also, adding parser support for things we do not actually support in layout would add extra burden as we may need a separate pref to control its exposing if we decide to expose e.g. shape-outside by default without support of shape-margin. And that usually also adds more documentation work with no good reason, and that could even confuse developers.
I'll try to move this forward.
Assignee: kuoe0.tw → bwerth
This will be resolved as part of the implementation, in Bug 1265342.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.