Closed
Bug 1006326
Opened 11 years ago
Closed 11 years ago
Enable 'box-decoration-break' by default
Categories
(Core :: CSS Parsing and Computation, enhancement)
Core
CSS Parsing and Computation
Tracking
()
VERIFIED
FIXED
mozilla33
Tracking | Status | |
---|---|---|
firefox30 | --- | unaffected |
firefox31 | --- | unaffected |
firefox32 | + | verified |
firefox33 | --- | verified |
relnote-firefox | --- | 32+ |
People
(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
1.02 KB,
patch
|
roc
:
review+
lmandel
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
No description provided.
What blocks enabling it on all builds (Nightly + Release)?
Flags: needinfo?(matspal)
Comment 2•11 years ago
|
||
I probably think it requires a tiny bit more testing.
Updated•11 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Comment 3•11 years ago
|
||
The remaining piece is to implement layout of box-decoration-break:clone
for block margins: bug 1025669. I don't know if that needs to block
enabling it though. In particular, b-d-b should be fully working for
inlines as a replacement for -moz-background-inline-policy.
Flags: needinfo?(mats)
Comment 4•11 years ago
|
||
Is there a particular reason this should only be enabled for Nightly, or can we enable it and let it ride the trains?
Assignee | ||
Comment 5•11 years ago
|
||
The implementation should be complete now, except for clip-path/clip/mask
in bug 988659. Do we want to block on that bug?
Flags: needinfo?(dbaron)
I think we do *not* want to block, though I don't have a good sense of how hard it is. (My guess is that it's hard-ish.)
Flags: needinfo?(dbaron)
Assignee | ||
Comment 7•11 years ago
|
||
Looks green so far...
https://tbpl.mozilla.org/?tree=Try&rev=ee02ec9cf863
https://tbpl.mozilla.org/?tree=Try&rev=a7c6113b1fb4
Assignee: nobody → mats
Attachment #8451167 -
Flags: review?(roc)
Assignee | ||
Comment 8•11 years ago
|
||
OK, so for FF32 I would recommend we uplift bug 1025669 (block margins),
bug 1031444 (bug fix) and bug 1031726 (clip:rect()) and then enable it
by default there too.
Does that seem reasonable?
Flags: needinfo?(dbaron)
Attachment #8451167 -
Flags: review?(roc) → review+
Assignee | ||
Comment 9•11 years ago
|
||
Assignee | ||
Comment 10•11 years ago
|
||
BTW, should we post an "intent to ship" about this property?
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Comment 12•11 years ago
|
||
(In reply to Mats Palmgren (:mats) from comment #10)
> BTW, should we post an "intent to ship" about this property?
Yes I believe we should do that with all new Web-exposed features.
Assignee | ||
Comment 13•11 years ago
|
||
OK, will do so as soon as dbaron responds to the n-i in comment 8.
The patches for Aurora are green on Try, fwiw:
https://tbpl.mozilla.org/?tree=Try&rev=98bad8e3d58f
(In reply to Mats Palmgren (:mats) from comment #8)
> OK, so for FF32 I would recommend we uplift bug 1025669 (block margins),
> bug 1031444 (bug fix) and bug 1031726 (clip:rect()) and then enable it
> by default there too.
>
> Does that seem reasonable?
It's a little bit of a stretch of the purpose of Aurora, but I think it seems reasonable given that it avoids having the feature disappear for one release and then come back with a new name in the following one (right?).
Flags: needinfo?(dbaron)
Assignee | ||
Comment 15•11 years ago
|
||
Right, the uplift is intended to avoid that. Great, so I'll request
approval on the patches in the relevant bugs as usual then.
Assignee | ||
Comment 16•11 years ago
|
||
Comment on attachment 8451167 [details] [diff] [review]
fix
Approval Request Comment
[Feature/regressing bug #]:
We want to enable 'box-decoration-break' by default on Aurora
to provide an upgrade path for a property that were removed
(-moz-background-inline-policy).
[User impact if declined]:
Web pages that use the old property will have some element's
background image displayed slightly wrong, and there
will be one cycle without an upgrade path to fix that.
[Describe test coverage new/current, TBPL]:
It's enabled by default on m-c.
Most of the 'box-decoration-break' patches are already on
Aurora but the feature is currently disabled by default.
[Risks and why]: minor risk
[String/UUID change made/needed]: none
Attachment #8451167 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•11 years ago
|
Summary: Enable 'box-decoration-break' by default in Nightly → Enable 'box-decoration-break' by default
Updated•11 years ago
|
status-firefox30:
--- → unaffected
status-firefox31:
--- → unaffected
status-firefox32:
--- → affected
status-firefox33:
--- → fixed
tracking-firefox32:
--- → +
relnote-firefox:
--- → +
Comment 17•11 years ago
|
||
Comment on attachment 8451167 [details] [diff] [review]
fix
Aurora+
Attachment #8451167 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 18•11 years ago
|
||
Release note should include that box-decoration-break replaces -moz-background-inline-policy, which has been removed from Firefox 32.
Mats - Can you write a draft release note about this change? Is the following correct? If this is not a direct replacement it may be better to have a note about the removal and a note about the new property.
"-moz-background-inline-policy CSS property replaced with box-decoration-break"
As well, is the upgrade path documented somewhere that we can link to from the release notes?
Updated•11 years ago
|
Flags: needinfo?(mats)
Assignee | ||
Comment 19•11 years ago
|
||
I think the text on the MDN page might work as guidance for authors:
https://developer.mozilla.org/en-US/docs/Web/CSS/-moz-background-inline-policy
"This property has been standardized in css3-background with the name box-decoration-break, except that box-decoration-break applies to borders in addition to backgrounds. The continuous value is equivalent to slice, and the each-box value is equivalent to clone. An equivalent to the bounding-box value was not included in the standard, and the functionality is likely to be removed when the standard feature is implemented."
The spec has changed since then, it's css-break, not css3-background.
http://dev.w3.org/csswg/css-break/#break-decoration
We should also update the current MDN page for b-d-b, which looks out of date:
https://developer.mozilla.org/en-US/docs/Web/CSS/box-decoration-break
(and link to it from the former page)
Flags: needinfo?(mats)
Assignee | ||
Comment 20•11 years ago
|
||
(btw, I'm updating those MDN pages a bit, while watching the tree...)
Assignee | ||
Comment 21•11 years ago
|
||
Assignee | ||
Comment 22•11 years ago
|
||
I have updated the MDN pages. I'd like to add a more prominent "obsolete"
warning on the -moz-background-inline-policy page but I don't know how to do
that. I tried to link the CSS spec "CSS Fragmentation Module Level 3" on
the box-decoration-break page but MDN doesn't seem to recognize it as
a valid spec name and I don't know how to make it one. (I've added
a plain external link there for now.)
Flags: needinfo?(jypenator)
Comment 23•11 years ago
|
||
Mats, I've added enough "warnings" on -moz-b-i-p I think. I also fixed the link on box-decoration-box and made slight modification to match our look and feel. I updated the text on Fx 32 for devs too.
I think we have an up-to-date doc for this property.
Mats, thanks for the very valuable help! Nice examples on the property doc!
Flags: needinfo?(jypenator)
Keywords: dev-doc-needed → dev-doc-complete
Updated•11 years ago
|
Comment 24•11 years ago
|
||
Mats, I've tested using three samples from https://developer.mozilla.org/en-US/docs/Web/CSS/box-decoration-break using Firefox 32 beta 8 and latest Aurora (20140819004001) under Win 7 64-bit, Ubuntu 13.04 32-bit and Mac OSX 10.9.4.
Everything looks good except for the box-decoration-break-inline-extreme under Mac, where the characters are exceeding the circles. Please see http://i.imgur.com/O4IbnFa.png.
Is this related to this issue? Thanks
Flags: needinfo?(mats)
Assignee | ||
Comment 25•11 years ago
|
||
(In reply to Petruta Rasa [QA] [:petruta] from comment #24)
> Everything looks good except for the box-decoration-break-inline-extreme
> under Mac, where the characters are exceeding the circles. Please see
> http://i.imgur.com/O4IbnFa.png.
Yeah, I think that can happen with extreme border-radius values.
I guess we can avoid that and make it look better by adding some
padding, say padding:10px. Feel free to update the example and
add a new screenshot on the wiki-page ;-)
Flags: needinfo?(mats)
Comment 26•11 years ago
|
||
Adding padding:5px makes it look better on Mac too, thanks.
Marking as verified as per comment 24.
Updated•11 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•