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

[WIP] improve non-basic $url modifier description #256

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

stanislav-atr
Copy link
Contributor

No description provided.

@@ -3386,7 +3386,8 @@ The [special characters](#basic-rules-special-characters) and [regular expressio

:::info Compatibility

Rules with the `$url` modifier are supported by AdGuard for Windows, Mac, and Android, **running CoreLibs version 1.11 or later**, and AdGuard Browser Extension, **running TSUrlFilter vX.X.X or later**.
* Rules with the `$url` modifier are supported by AdGuard for Windows, Mac, and Android, **running CoreLibs version 1.11 or later**, and AdGuard Browser Extension, **running TSUrlFilter vX.X.X or later**.
* `url` modifiers is not compatible with `$domain` and `$path` modifiers and domain-specific rules. For example, the rule `[$url=/category/*]example.com###textad` will be rejected.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sfionov Does url modifier for cosmetic rules behave this way in corelibs?

@slavaleleka slavaleleka changed the title [WIP] update cosmetic url modifier compatibility [WIP] update non-basic $url modifier compatibility Jul 18, 2024
@slavaleleka slavaleleka changed the title [WIP] update non-basic $url modifier compatibility improve non-basic $url modifier description Jul 18, 2024
@slavaleleka slavaleleka changed the title improve non-basic $url modifier description [WIP] improve non-basic $url modifier description Jul 19, 2024
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.

None yet

2 participants