Skip to content

fix: remove :-moz-placeholder from :placeholder-shown property in Firefox > 50 #1535

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

Marukome0743
Copy link
Contributor

Description

Fix #1533.

I have changed the behavior of :placeholder-shown at #1532.
However, this introduced the bug with :not() pseudo-class.

For example, see the comment.

To solve this issue, :-moz-placeholder is given only for Firefox under 50 version.
Furthermore, I updated the placeholder-shown.out.css test.

@Marukome0743 Marukome0743 marked this pull request as draft March 21, 2025 12:48
@Marukome0743 Marukome0743 marked this pull request as ready for review March 21, 2025 15:25
@Marukome0743 Marukome0743 marked this pull request as draft March 21, 2025 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

:placeholder-shown is replaced with -moz-placeholder, although this is outdated
1 participant