Skip to content

Fix #239237 www.nikkei.com - #239250

Merged
Alex-302 merged 13 commits into
masterfrom
Fix-#239237-www.nikkei.com
Sep 3, 2026
Merged

Alex-302 merged 13 commits into
masterfrom
Fix-#239237-www.nikkei.com

Conversation

@mu-hun

@mu-hun mu-hun commented Aug 25, 2026

Copy link
Copy Markdown
Member

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

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 review from Alex-302 and Yuki2718 August 25, 2026 06:58
@mu-hun mu-hun self-assigned this Aug 25, 2026
Comment thread SocialFilter/sections/specific.txt Outdated
Comment thread SocialFilter/sections/specific.txt Outdated
baseball.yahoo.co.jp##.bb-centerColumn > #inf:has(> .sn-modNotice > .sn-notice--annapurna)
nikkei.com##div[data-rn-inview-track="hub-ad-recommend"]
nikkei.com##div[class^="trendWatchContainer_"]
nikkei.com##body > div[class^="adsArea_"]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Too wide. Did you test it?

Image

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes. I tested It and It was intended. I will recheck tomorrow for narrowing rules 👀

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yeah, that hides too much. Also neither of the above two rules (##div[data-rn-inview-track="hub-ad-recommend"], ##div[class^="trendWatchContainer_"] hits on my end, either on mobile or on desktop. On my end "トレンドウォッチ" is ##div[class^="trendWatchList_"] and what's reported will require something like ##div[class^="item_"]:has(> a[class^="commonStylesAnchor_"][href*="/?utm_source="]).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

On my end other rules applied on another pages.

@Yuki2718 Yuki2718 Aug 25, 2026 •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Okay, found them on the top page, but hub-ad-recommend can be added to the Japanese filter. I'll add it. but ##div[data-rn-inview-track="hub-ad-recommend"] hides too much on mobile. Maybe something like ##div[data-rn-track="hub-ad-recommend"]:has(> a[href*="/?utm_source="]).

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The ##div[class^="trendWatchContainer_"] container is still make sense. because It incudes self-promotion contents. By the way It has incorrect blocking in AdGuard. I'm checking this issue now.

Adbocker Enabled / Disabled Screenshot 2026-08-26 at 1 57 34 PM Screenshot 2026-08-26 at 1 57 54 PM

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've found the reason. The self-promo content is served under nkis.nikkei.com, which is blocked by adserver.txt in JapaneseFilter.

I have listed self-promotion requests below, but they didn't work due to ||nkis.nikkei.com.

! nikkei.com self-promo widget [data-kad-id="5226_7151"][data-kad-type="trendwatch"], confirmed serving content
@@||nkis.nikkei.com/ad?*section_id=7151&*$domain=nikkei.com
@@||nkis.nikkei.com/ad?*section_id=7153&*$domain=nikkei.com
@@||nkis.nikkei.com/ad?*section_id=10810&*$domain=nikkei.com
@@||nkis.nikkei.com/ad?*section_id=7155&*$domain=nikkei.com
!
! nikkei.com self-promo widget [data-kad-id="5225_7149"][data-kad-type="infeed-wide"]
@@||nkis.nikkei.com/ad?*section_id=7149&*$domain=nikkei.com

Request URL example: nkis.nikkei.com/ad?ord=6724574257231&section_id=7149&handler_name=process_tag_provider_response_7149

What should I do about this issue?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

In this case it's okay to keep them blocked. It requires too many exceptions, and sometimes we just let recommendations blocked than adding exceptions - other times not and all these depend on various factors.

Comment thread AnnoyancesFilter/Cookies/sections/cookies_specific.txt Outdated
Comment thread AnnoyancesFilter/Other/sections/self-promo.txt Outdated
Comment thread SocialFilter/sections/specific.txt Outdated
Comment thread AnnoyancesFilter/Popups/sections/subscriptions_specific.txt Outdated
Comment thread AnnoyancesFilter/Other/sections/self-promo.txt
Comment thread SocialFilter/sections/specific.txt Outdated
@mu-hun
mu-hun requested review from Alex-302 and Yuki2718 September 2, 2026 08:23
Comment thread SocialFilter/sections/specific.txt Outdated
@mu-hun

mu-hun commented Sep 3, 2026

Copy link
Copy Markdown
Member Author

@Alex-302 Please merge this PR 👀

@Alex-302
Alex-302 merged commit 93e2f69 into master Sep 3, 2026
2 checks passed
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.

3 participants