Remove wrong placeholder-shown prefixing for Firefox#1545
Conversation
|
Have you just break all I know about this problem, but we need careful solution. I would like to accept PR, but I need from you to think about all edge cases. |
|
Do you know that it actually works in Firefox < 50? Because I couldn't find any proof for it. But even if it does, it should only be activated for Firefox < 50, as it's not needed in modern version and even breaks some selectors, like shown in my example🙈 |
It is mark as supported in Can I Use https://caniuse.com/?search=placeholder-shown
We are adding it because of KaiOS 2.5 (FF based browser for cheap phones very popular in Afrika). It is part of default browser set. |
|
That explains why adding |
Replacing
:placeholder-shownwith:-moz-placeholderbreaks in Firefox. While placeholder-shown is working fine (supported since Jan. 2017), the:-moz-placeholderadded by autoprefixer breaks selectors with:not:Example:
produces:
This always applies, even if the placeholder is shown, because
:not(:-moz-placeholder)evaluates to true.I couldn't find any resources stating that
:-moz-placeholderbehaves like:placeholder-shown, so I wonder why it's in here?