-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add entry for input-security
CSS property
#12822
Conversation
113f701
to
4c11c78
Compare
4c11c78
to
b3cb22f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR should remove the -webkit-text-security file (and add it to input-security as and old alias).
https://github.com/mdn/browser-compat-data/blob/main/css/properties/-webkit-text-security.json
See also mdn/content#10438
It's not actually an alias of that property, as can be seen by its WebKit implementation. |
I asked in mdn/content#10438 if we want one feature page here or if we should keep them separate. |
Removing my review request. Need info as requested in mdn/content#10438 (comment) |
Marking as "not ready" since this seems to be blocked on info needed for the content PR. |
Looks like Safari has put this behind a flag for now WebKit/WebKit@bed3cc8 |
And sounds like the CSSWG might drop the property from the spec entirely. w3c/csswg-drafts#6788 |
Given the above two points, and the lack of movement on the blocking discussion for this PR, I'm gonna go ahead and close it for now. |
Summary
Added an entry for the
input-security
CSS property.Test results and supporting details
https://webkit.org/blog/11975/release-notes-for-safari-technology-preview-133/#:~:text=Added%20support%20for%20input-security - Safari TP
supposedly supports this property but I and someone else haven't been able to get it working so have left as false for now.As of TP 134 this is now working so I've added it.WebKit bug (implemented): https://webkit.org/b/184510
Firefox bug (in progress): https://bugzil.la/1735384
Chromium bug: https://crbug.com/1253732
Related issues
Fixes #12664
Related to mdn/content#9422 and mdn/content#10438