Skip to content

Conversation

@rakleed
Copy link
Contributor

@rakleed rakleed commented Apr 26, 2025

Prerequisites

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

Add your comment and screenshots

  1. Your comment

Share buttons on header and comments on Substack-based sites

  1. Screenshots
Screenshot 1:

image

Terms

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

@rakleed rakleed changed the title Fix share buttons on Substack-based site Fix share buttons on Substack-based sites Apr 26, 2025
@Alex-302 Alex-302 added the A: In progress Work on the issue is in progress label May 6, 2025
@Alex-302 Alex-302 requested a review from Copilot May 6, 2025 12:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request updates the share button rules to fix issues on Substack-based sites and ensure more comprehensive coverage across various domains.

  • Refactored domain lists to consolidate multiple sites into single rules
  • Removed duplicate or overly specific rules for Substack and related domains

boosty.to##div[class^="PostFooter-scss--module_share"]
lsm.lv##.social-icons-block
gamefile.news##span > a.pencraft:has(> div.pencraft > svg.lucide-share)
chipsandcheese.com,noahpinion.blog,newcomer.co,blog.corkmac.app,blog.sshh.io,hardcoresoftware.learningbyshipping.com,newsguardrealitycheck.com,newsletter.techworld-with-milan.com,newsletter.pragmaticengineer.com,substack.com,gamefile.news##span > a.pencraft:has(> div.pencraft > svg.lucide-share)

This comment was marked as outdated.

publico.es##li[data-modal-target="share-modal"]
vistazo.com##.rrss
transformernews.ai##button[aria-label="View more"]:has(> svg.lucide-share)
chipsandcheese.com,noahpinion.blog,newcomer.co,blog.corkmac.app,blog.sshh.io,hardcoresoftware.learningbyshipping.com,newsguardrealitycheck.com,newsletter.techworld-with-milan.com,newsletter.pragmaticengineer.com,usermag.co,theolognion.com,substack.com,transformernews.ai##button[aria-label="View more"]:has(> svg.lucide-share)

This comment was marked as outdated.

@Alex-302 Alex-302 merged commit e27889e into AdguardTeam:master May 6, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: In progress Work on the issue is in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants