Re: Intent to implement: CSS {background,mask}-repeat-{x/y} properties

2016-12-15 Thread Tommy Kuo
Hi Mike,  There is a site[1] using these properties. And the related bug is here[2]. [1]: http://ocando.vnz.la/ [2]: https://bugzilla.mozilla.org/show_bug.cgi?id=1273244 --  Tommy Kuo / Software Engineer ku...@mozilla.com Mozilla Taiwan On December 16, 2016 at 05:18:36, Mike Taylor

Re: Intent to implement: CSS {background,mask}-repeat-{x/y} properties

2016-12-15 Thread Mike Taylor
Hi Tommy, On 11/27/16 9:59 PM, Tommy Kuo wrote: Currently, for web compatibility, I think we should implement these properties. Do we know about any sites that are broken due to background-repeat-x/y? (apologies in advance if you link to a bug that has me commenting on it...) -- Mike Taylor

Re: Intent to implement: CSS {background,mask}-repeat-{x/y} properties

2016-11-28 Thread Tommy Kuo
I think we should only implement the `background-repeat-{x|y}`. Currently, no browser engine supports `mask-repeat-{x|y}`. But for `background-repeat-{x|y}`, Google Chrome already supports it. In this comment[1], the usage of these properties is above the deprecation threshold (both usage rate

Re: Intent to implement: CSS {background,mask}-repeat-{x/y} properties

2016-11-27 Thread Tommy Kuo
There is a mistake on last mail. The following description should be corrected. > The behavior of `background-repeat-x: no-repeat; background-repeat-y: > no-repeat;` is repeat the background only on x-axis, like `background-repeat: > no-repeat no-repeat`. It should be "The behavior of

Re: Intent to implement: CSS {background,mask}-repeat-{x/y} properties

2016-11-27 Thread Tommy Kuo
I think we should only implement the `background-repeat-{x|y}`. Currently, no browser engine supports `mask-repeat-{x|y}`. But for `background-repeat-{x|y}`, Google Chrome already supports it. In this comment[1], the usage of these properties is above the deprecation threshold (both usage rate

Re: Intent to implement: CSS {background,mask}-repeat-{x/y} properties

2016-11-24 Thread L. David Baron
On Friday 2016-11-25 02:09 +0800, Chih-Hsuan Kuo wrote: > It means the browser engines can use the properties, but these properties > don't accept any value. It also means the properties don't work. > > > In Google Chrome, these properties can be auto-completed. And the error shows > when we

Re: Intent to implement: CSS {background,mask}-repeat-{x/y} properties

2016-11-24 Thread Chih-Hsuan Kuo
It means the browser engines can use the properties, but these properties don't accept any value. It also means the properties don't work. In Google Chrome, these properties can be auto-completed. And the error shows when we set the value to these properties. In Safari, there is no warning

Re: Intent to implement: CSS {background,mask}-repeat-{x/y} properties

2016-11-24 Thread L. David Baron
On Thursday 2016-11-24 17:28 +0800, Tommy Kuo wrote: > Sorry, I think some people can not see the table of "status on other browser > engines". The plain text version is following: > > background-repeat-{x,y} > > Google Chrome: Shipped > Safari: Only property name can be recognized > Edge: No

Intent to implement: CSS {background,mask}-repeat-{x/y} properties

2016-11-24 Thread Tommy Kuo
Summary In this work, I’d like to implement 4 new longhand properties: background-repeat-x, background-repeat-y, mask-repeat-x, and mask-repeat-y. And it also makes background-repeat and mask-repeat become shorthand properties. Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1273244 Link to