Skip to content

Fix #227109 endfield.gryphline.com Annoyances – cookie notice#227111

Open
mu-hun wants to merge 2 commits intomasterfrom
fix-#227109-endfield.gryphline.com
Open

Fix #227109 endfield.gryphline.com Annoyances – cookie notice#227111
mu-hun wants to merge 2 commits intomasterfrom
fix-#227109-endfield.gryphline.com

Conversation

@mu-hun
Copy link
Member

@mu-hun mu-hun commented Mar 16, 2026

To avoid invalid pull requests, please check and confirm following terms

  • This is not an ad/bug report;
  • My code follows the guidelines and syntax of this project;
  • I have performed a self-review of my own changes;
  • My changes do not break web sites, apps and files structure.

What problem does the pull request fix?

If the problem does not fall under any category that is listed here, please write a comment below in corresponding section

  • Missed ads or ad leftovers;
  • Website or app doesn't work properly;
  • AdGuard gets detected on a website;
  • Missed analytics or tracker;
  • Social media buttons — share, like, tweet, etc;
  • Annoyances — pop-ups, cookie warnings, etc;
  • Filters maintenance.

What issue is being fixed?

Enter the issue address

Fix #227109 endfield.gryphline.com Annoyances – cookie notice

Screenshot 1

Terms

  • By submitting this issue, I agree that pull request does not contain private info and all conditions are met

@mu-hun mu-hun requested a review from a team March 16, 2026 03:44
@mu-hun mu-hun self-assigned this Mar 16, 2026
!
! NOTE: Regular rules
!
endfield.gryphline.com##body > div:has(a[href*="/cookiesPolicy"])
Copy link
Member Author

@mu-hun mu-hun Mar 16, 2026

Choose a reason for hiding this comment

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

This website's cookie notice lacks a unique selector name. It can only be solved with the :has selector.

Another way to resolve it is with a scriptlet—'set-cookie'. Which method is preferred here? 🤔

endfield.gryphline.com#%#//scriptlet('set-cookie', 'cookieAccepter', '1')

Copy link
Member Author

@mu-hun mu-hun Mar 16, 2026

Choose a reason for hiding this comment

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

This website's cookie notice lacks a unique selector name. It can only be solved with the :has selector.

Another way to resolve it is with a scriptlet—'set-cookie'. Which method is preferred here? 🤔

@piquark6046 I would like to hear your opinion on this case 👀

Copy link
Member

Choose a reason for hiding this comment

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

I think that the current cosmetic filter is ok.

@mu-hun mu-hun requested a review from piquark6046 March 16, 2026 04:55
piquark6046
piquark6046 previously approved these changes Mar 16, 2026
!
! NOTE: Regular rules
!
endfield.gryphline.com##body > div:has(a[href$="/cookiesPolicy"])
Copy link
Member

Choose a reason for hiding this comment

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

div:has(a is dangerous. It may hide more than necessary if the nesting of elements changes (which sometimes happens)

Better endfield.gryphline.com##body > div[class*=" "]:has(> div > span > a[href*="/cookiesPolicy"])

Also I see a cookie notification on the main page. Check another subdomains.

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.

endfield.gryphline.com

3 participants