Makoto Kato [:m_kato] Reporter
	Description • 10 years ago

WebKit/Blink and Trindent are implemented, but it has no property in Gecko.

	Mats Palmgren (inactive)
	Comment 1 • 10 years ago

(Bugs about implementing new CSS properties are usually filed in "CSS
Parsing and Computation", so moving it there for consistency.)

	David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)
	Comment 2 • 10 years ago

(In reply to Mats Palmgren (:mats) from comment #1)
> (Bugs about implementing new CSS properties are usually filed in "CSS
> Parsing and Computation", so moving it there for consistency.)

Really?  I'd keep it in the component where the bulk of the code would live.  And the CSS side here is trivial.

	Mats Palmgren (inactive)
	Comment 3 • 10 years ago

My mistake then;  I thought it was the established practice for bugs that
adds new CSS properties.

	Makoto Kato [:m_kato] Reporter
	Comment 4 • 9 years ago

Although, I am migrating ICU into m-c (by some bugs for ECMA-402 support), ICU has this line-break rule now
http://www.icu-project.org/trac/changeset/37037

	Koji Ishii
	Comment 5 • 7 years ago

FYI, Blink is going to unprefix 'line-break' property (actually, we had, accidentally, now trying to be official.)

mkwst@ suggested we should update our status here, hope this info can be any help for you.

Intent to Ship: Unprefix CSS 'line-break' property
https://groups.google.com/a/chromium.org/d/msg/blink-dev/RHeWSxMIVK8/g5NJgZo9AQAJ

	Boris Zbarsky [:bzbarsky]
	Comment 6 • 7 years ago

It would be good to not have this random incompatibility.

Makoto-san, do you happen to have an idea of how much work this is to implement?

	Makoto Kato [:m_kato] Reporter
	Comment 7 • 7 years ago

(In reply to Boris Zbarsky [:bz] (vacation until 7/7) from comment #6)
> It would be good to not have this random incompatibility.
> 
> Makoto-san, do you happen to have an idea of how much work this is to
> implement?

If we can keep our current line breaker rule that isn't same as UAX#14, it isn't difficult.  If line-break property need additional rule again our rule.  If we require this on 57 or 58, I will start the implementation.

If we have to change it to UAX#14 based, it isn't easy.  One is file size (+3MB with ICU's rule), and others are compatibility  against old Gecko.  File size is critical for mobile team and we don't use ICU on Fennec's release channel yet.

	Boris Zbarsky [:bzbarsky]
	Comment 8 • 7 years ago

> If we require this on 57 or 58, I will start the implementation

I don't know that we have a target for this yet.  57 is probably not realistic no matter what, but "sometime in the nearish future" might be nice if everyone else is shipping this....

	Kan-Ru Chen [:kanru] (UTC+9)
	Comment 9 • 7 years ago

I wonder if it's possible to base this on the xi-unicode library that Servo is also using to implement line breaking. Looks like bug 1290022 is investigating the possibility.

	Pulsebot
	Comment 17 • 5 years ago

Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e7b525e15531
Add parsing of the loose|normal|strict values of the CSS line-break property. r=emilio
https://hg.mozilla.org/integration/autoland/rev/f339909acbc3
Pass word-break and line-break properties to the line-breaker as two distinct enum class parameters. r=m_kato
https://hg.mozilla.org/integration/autoland/rev/436e3199c386
Implement loose|normal|strict line-break values by remapping the linebreak class of affected characters. r=m_kato
https://hg.mozilla.org/integration/autoland/rev/2607e7773ed5
Restrict some line-break behaviors to Chinese and Japanese contexts, as per spec. r=m_kato
https://hg.mozilla.org/integration/autoland/rev/b695489a68c4
Reduce the available width slightly in line-break-loose-017a because a couple of the target characters may be less then .2em wide. r=m_kato
https://hg.mozilla.org/integration/autoland/rev/43952b7f9de8
Update metadata for WPT tests that now pass. r=m_kato

	Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)
	Comment 18 • 5 years ago

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/16924 for changes under testing/web-platform/tests

	Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)
	Comment 19 • 5 years ago

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

	Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)
	Comment 21 • 5 years ago

Upstream PR merged