Re: Intent to unship: -moz-placeholder pseudo-element and pseudo-class

2017-05-25 Thread Mike Taylor 
On 5/25/17 5:48 AM, Ku(顧思捷)CJ wrote:
> Mike, do we get any complaint about not supporting webkit placeholder
> alias?  

Nah, not that I'm aware of. I was just looking through GitHub search
results and finding some stuff that only includes webkit-input-placeholder:



Pretty much everywhere else that has a -moz-placeholder includes a
-webkit-input-placeholder (or unprefixed, or both). But I'm guessing
add-on code would be the exception.

-- 
Mike Taylor
Web Compat, Mozilla
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Re: Intent to unship: -moz-placeholder pseudo-element and pseudo-class

2017-05-24 Thread Boris Zbarsky 

On 5/24/17 1:06 PM, Mike Taylor wrote:

[1]



Sadly, that code is already buggy in Firefox: it uses 
":moz-placeholder", which doesn't parse.  The thing that parses is 
":-moz-placeholder".


-Boris

___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Re: Intent to unship: -moz-placeholder pseudo-element and pseudo-class

2017-05-24 Thread Mike Taylor 
Hi CJ,

On 5/23/17 10:13 PM, Ku(顧思捷)CJ wrote:
> I intend to remove "-moz-placeholder" pseudo-element and pseudo-class in
> bug 1300896.
> 
> We already supported canonical version of them:
> 1. "::placeholder" in bug 1069012, FF 51.
> 2. ":placeholder-shown" in bug 1069015, FF 51.
> 
> To support these mozilla-specific aliases introduces special-case handling
> in both stylo and gecko, which bring in unnecessary complexity.

It's pretty easy to find code on GitHub where only moz/webkit/ms
prefixes are used[1]. How complex is the special-case handling, beyond a
simple prefixed <-> unprefixed alias?

[1]



-- 
Mike Taylor
Web Compat, Mozilla
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Re: Intent to unship: -moz-placeholder pseudo-element and pseudo-class

2017-05-24 Thread Gijs Kruitbosch 

On 24/05/2017 03:13, Ku(顧思捷)CJ wrote:

I intend to remove "-moz-placeholder" pseudo-element and pseudo-class in
bug 1300896.


Which will ship in which release? 55 or 56? Esp. asking given the 
upcoming feature freeze for 55.


~ Gijs
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Intent to unship: -moz-placeholder pseudo-element and pseudo-class

2017-05-23 Thread 顧思捷 
I intend to remove "-moz-placeholder" pseudo-element and pseudo-class in
bug 1300896.

We already supported canonical version of them:
1. "::placeholder" in bug 1069012, FF 51.
2. ":placeholder-shown" in bug 1069015, FF 51.

To support these mozilla-specific aliases introduces special-case handling
in both stylo and gecko, which bring in unnecessary complexity.
___
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform