Astley Chen (inactive) Assignee
	Description • 8 years ago

Per bug 859301 comment 27, it's better to leave :-moz-dir() for a short transition period rather than removing it at exactly the same time as adding :dir.
Open this bug to track the removal status.

	Mike Taylor [:miketaylr]
	Comment 1 • 8 years ago

Can we do some research to see how widely used :-moz-dir() is (and if sites using it include an unprefixed fallback) before removing it?

	j.j.
	Comment 2 • 8 years ago

Adding Blink and WebKit bugs to "See Also" field

	Xidorn Quan [:xidorn] UTC+11
	Comment 3 • 8 years ago

I don't really believe this is widely-used currently. And even if it is, given that neither WebKit/Blink nor IE/Edge support it, I don't think removing the unprefixed version would break websites badly.

I guess the best way to research this is to query the httparchive on the BigQuery. Hopefully they have CSS files included.

	Astley Chen (inactive) Assignee
	Comment 7 • 8 years ago

TRY: https://treeherder.mozilla.org/#/jobs?repo=try&revision=901949ce5bcfb9a737ff30dcb469a71d87fc1264

Unprefix version had been shipped on FF49, it's time to remove the support of prefix one from FF53.
We will also take care of any webcompat issues jump out due to this change, and make the call whether or not worth doing it.

	Xidorn Quan [:xidorn] UTC+11
	Comment 8 • 8 years ago
mozreview-review

Comment on attachment 8813209 [details]
Bug 1270406 - Remove :-moz-dir support.

https://reviewboard.mozilla.org/r/94716/#review95146

	Patrick Brosset <:pbro>
	Comment 9 • 8 years ago
mozreview-review

Comment on attachment 8813208 [details]
Bug 1270406 - Unprefix pseudo-class dir in DevTool resources.

https://reviewboard.mozilla.org/r/94714/#review95202

	Drew Willcoxon :adw
	Comment 10 • 8 years ago
mozreview-review

Comment on attachment 8813207 [details]
Bug 1270406 - Unprefix pseudo-class dir in browser resources.

https://reviewboard.mozilla.org/r/94712/#review95404

	Astley Chen (inactive) Assignee
	Comment 11 • 8 years ago

Thanks for your review.

	Pulsebot
	Comment 12 • 8 years ago

Pushed by tlin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/96be179dc002
Unprefix pseudo-class dir in browser resources. r=adw
https://hg.mozilla.org/integration/autoland/rev/71620574d3e5
Unprefix pseudo-class dir in DevTool resources. r=pbro
https://hg.mozilla.org/integration/autoland/rev/1ba616d48ed7
Remove :-moz-dir support. r=xidorn

	Carsten Book [:Tomcat]
	Comment 13 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/96be179dc002
https://hg.mozilla.org/mozilla-central/rev/71620574d3e5
https://hg.mozilla.org/mozilla-central/rev/1ba616d48ed7

	Kohei Yoshino [:kohei]
	Comment 14 • 8 years ago

Posted the site compatibility doc: https://www.fxsitecompat.com/en-CA/docs/2016/moz-dir-pseudo-class-has-been-removed/

	Sebastian Zartner [:sebo]
	Comment 15 • 8 years ago

Mentioned the removal at https://developer.mozilla.org/en-US/docs/Web/CSS/:dir#Browser_compatibility and https://developer.mozilla.org/en-US/Firefox/Releases/53.

Sebastian