-
Notifications
You must be signed in to change notification settings - Fork 52
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
fix(helper): .sr-only
clip
#297
Conversation
✅ Deploy Preview for vuepress-ecosystem ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Pull Request Test Coverage Report for Build 12210710363Details
💛 - Coveralls |
Hi @Mister-Hope, I think |
@labuladong I tried that in WeChat, but it worked fine. It seems that my WeChat is using Chromium 130. (test site) |
Oh, it failed in my WeChat and reproduced with https://developers.weixin.qq.com/miniprogram/dev/devtools/download.html My workaround is adding this CSS manually. |
There's another solution - just remove |
Update here, I tried WeChat devetools and reproduce what you are reporting: It does say it does not support clip-path, but it's using chrome 86. According to https://caniuse.com/mdn-css_properties_clip-path_basic_shape and real test on a chrome 55 with playwright, everything is working as expected. Maybe you should raise an issue to WeChat to team to as why this property is not supported as a real chrome 86 does. BTW try to reproduce yourself in a chrome 86 first. |
Before submitting the PR, please make sure you do the following
close #123
).What is the purpose of this pull request?
Description
clip
has been deprecated, andrect (0, 0, 0, 0)
should berect(0 0 0 0)