-
Notifications
You must be signed in to change notification settings - Fork 327
AG-27455 update the :has() pseudo-class section
#795
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
Conversation
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.
Looks good to me 🤙
💡 To request another review, post a new comment with "/windsurf-review".
|
Preview was deployed to: https://pull-request-795.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-795.kb-adg.pages.dev/ |
| even for rules with the `##` marker. | ||
| Currently, not all AdGuard products support the native implementation of `:has()` yet: | ||
| Currently, all modern AdGuard products support the native implementation of `:has()`: |
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.
странное уточнение, а какие не modern?
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.
более старые версии, не актуальные (последние)
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.
мошт ссылку на can-i-use прост вставим?
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.
а что за ссылка? хочется как-то прояснить здесь
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.
не, can-i-use — ето про браузеры, а тут типа про аг-продукты. короче, сделал проще, убрал "modern"
Co-authored-by: Helen <[email protected]>
|
Preview was deployed to: https://pull-request-795.kb-adg.pages.dev/ |
…se into feature/AG-27455
|
Preview was deployed to: https://pull-request-795.kb-adg.pages.dev/ |
AdguardTeam/AdguardBrowserExtension#2587