Closed Bug 953408 Opened 11 years ago Closed 9 years ago

Unprefix -moz-hyphens

Categories

(Core :: CSS Parsing and Computation, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed
relnote-firefox --- 43+

People

(Reporter: sjw+bugzilla, Assigned: jtd)

References

(Blocks 3 open bugs)

Details

(Keywords: dev-doc-complete, site-compat)

Attachments

(2 files, 1 obsolete file)

Gecko has a very well working hyphenation and this part of the spec looks stable enough. So we should remove the prefix.
Blocks: unprefix
Depends on: 656750
Does the last call issues list for the spec look like it's not going to lead to hyphenation-related changes?
Keywords: dev-doc-needed
Last hyphenation-related changes were from 1 September 2011 to 19 January 2012: http://www.w3.org/TR/2012/WD-css3-text-20120119/#recent-changes Firefox supports the current spec. There are no hyphenation-related bugs or testcases: http://www.w3.org/Style/CSS/Tracker/products/10 http://test.csswg.org/suites/css3-text/nightly-unstable/html/chapter-6.htm
Severity: normal → enhancement
Component: Layout: Text → CSS Parsing and Computation
Blocks: css-text-3
Attached patch patch, unprefix -moz-hyphens property (obsolete) (deleted) — Splinter Review
Unprefix the -moz-hyphens property and add a temporary alias for -moz-hyphens ==> hyphens. Conversions of existing tests done in a separate patch.
Attachment #8660523 - Flags: review?(cam)
Assignee: nobody → jdaggett
Switch tests to use unprefixed hyphens property
Attachment #8660524 - Flags: review?(jfkthame)
This property and it's property value definitions have been stable for a while and implemented in prefixed form across IE/Webkit/Gecko for some time. Only Chrome lacks support for this. Our implementation includes support for a wide set of languages and I don't see any significant bug logged against hyphenation currently. I think we should do the unprefixing for the FF43 release.
Fix up minor omission in property_database.js, need to include alias explicitly.
Attachment #8660523 - Attachment is obsolete: true
Attachment #8660523 - Flags: review?(cam)
Attachment #8660552 - Flags: review?(cam)
(In reply to John Daggett (:jtd) from comment #5) > This property and it's property value definitions have been stable for a > while and implemented in prefixed form across IE/Webkit/Gecko for some time. > Only Chrome lacks support for this. > > Our implementation includes support for a wide set of languages and I don't > see any significant bug logged against hyphenation currently. I think we > should do the unprefixing for the FF43 release. Given all this, and that there have still been no changes to css-text-3 about hyphenation since the Last Call, and there are no open issues on it, I think it's safe to unprefix. Can you send an Intent to Ship mail to dev-platform about this? https://wiki.mozilla.org/WebAPI/ExposureGuidelines is geared towards DOM APIs, but we tend to use it for CSS features too (e.g. see Mats' display:contents Intent to Ship mail from a few months ago, for example). I don't expect anyone to have problems with this. I suggest mentioning in the mail that you plan to enable this at the end of this week, just to give people a few days to respond.
Attachment #8660552 - Flags: review?(cam) → review+
(In reply to Cameron McCormack (:heycam) from comment #8) > Can you send an Intent to Ship mail to dev-platform about this? > https://wiki.mozilla.org/WebAPI/ExposureGuidelines is geared towards DOM > APIs, but we tend to use it for CSS features too (e.g. see Mats' > display:contents Intent to Ship mail from a few months ago, for example). I > don't expect anyone to have problems with this. I suggest mentioning in the > mail that you plan to enable this at the end of this week, just to give > people a few days to respond. Yes, planning to do exactly this sometime this week!
Attachment #8660524 - Flags: review?(jfkthame) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Is it already to late for a relnote? Release Note Request (optional, but appreciated) [Why is this notable]: Notable change for WebDevs [Suggested wording]: Support for unprefixed 'hyphens' property [Links (documentation, blog post, etc)]: https://developer.mozilla.org/en-US/docs/Web/CSS/hyphens
relnote-firefox: --- → ?
Blocks: 656750
No longer depends on: 656750
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: