Skip to content
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

Update -webkit-text-security.json #19887

Merged
merged 6 commits into from
May 19, 2023
Merged

Update -webkit-text-security.json #19887

merged 6 commits into from
May 19, 2023

Conversation

andreydanil
Copy link
Contributor

-webkit-text-security is supported in Firefox 114: https://bugzilla.mozilla.org/show_bug.cgi?id=1826629.

Summary

whatwg/compat#240

Test results and supporting details

Tested on FF developer edition 114 and Firefox nightly 115.

Related issues

whatwg/compat#240

@github-actions github-actions bot added the data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS label May 17, 2023
Copy link
Contributor

@dipikabh dipikabh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only one nit. With the support added in Fx114, the following would also need to be updated on Ln 36:

"standard_track": false,

"standard_track": true,

@andreydanil andreydanil requested a review from dipikabh May 18, 2023 14:10
@andreydanil andreydanil requested a review from queengooborg May 19, 2023 15:35
Copy link
Contributor

@queengooborg queengooborg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, LGTM!

@queengooborg queengooborg merged commit 2281630 into mdn:main May 19, 2023
@andreydanil andreydanil deleted the patch-1 branch May 22, 2023 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants