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

google.com (Google Shopping Ads) #10132

Closed
6 tasks done
YoshiTabletopGamer opened this issue Oct 4, 2021 · 26 comments
Closed
6 tasks done

google.com (Google Shopping Ads) #10132

YoshiTabletopGamer opened this issue Oct 4, 2021 · 26 comments
Labels

Comments

@YoshiTabletopGamer
Copy link

YoshiTabletopGamer commented Oct 4, 2021

Prerequisites

I tried to reproduce the issue when...

  • uBO is the only extension
  • uBO with default lists/settings
  • using a new, unmodified browser profile

URL(s) where the issue occurs

https://www.google.com/search?q=smartphone&tbm=shop

Describe the issue

There are ads on the Shopping section of Google search. (I highlighted the ads with a red rectangle on the screenshot below) (The language of the text on the screenshot is Brazilian Portuguese)

Screenshot(s)

uBlock Origin version

1.38.2

Browser name and version

Firefox 92.0.1

Settings

  • Default

Notes

No response

@uBlock-user
Copy link
Contributor

uBlock-user commented Oct 4, 2021

Google Shopping page itself is an ad entirely, blocking it will break the intended purpose of the page.

@YoshiTabletopGamer
Copy link
Author

I argue that the part in the screenshot

that is not highlighted in red are not ads.

If the products I did not highlight with a red rectangle are also advertisements, how would they fit the EasyList policy? (Relevant parts in bold)

the definition of an advert as "the promotion of third party content in return for goods or services".

"... in return for goods or services"
An important question that has been raised is whether an advert requires the exchange of money, to which I would respond that it is not an essential element. Money may be one of the goods supplied, but there may also be services provided in return. Therefore a link to, say, a web host would be removed if this is a sponsored link required as a provision of service, but not if the description is merely factual, as this would not be promotion.

Google says that for the part I did not highlight with a red rectangle (Relevant parts in bold)

Unless otherwise indicated, items are ranked based on relevance, including your search terms and other Google activity. Learn more about managing how your experience is personalised. Some ads data, including your and other users’ interactions with ads, is used to improve the quality of the results. Google is not compensated for clicks into these results.

@uBlock-user

@uBlock-user
Copy link
Contributor

uBlock-user commented Oct 4, 2021

that is not highlighted in red are not ads.

They all pass through google.com/aclk? they're all product ads. You may believe and argue differently but that doesn't change the fact that buyers click on those links to buy products via google and blocking anything on that page is blocking functionality for that page.

That page is cosmetically exempted in uBO and in EasyList too. Same for Bing's shopping page.

@YoshiTabletopGamer
Copy link
Author

YoshiTabletopGamer commented Oct 4, 2021

How is this any different from blocking Amazon Sponsored Ads, for example?

buyers click on those links to buy products via amazon and blocking anything on that page is blocking functionality for that page.

I do understand your opinion, however. Thank you for maintaining the uBO filter lists.
@uBlock-user

@uBlock-user
Copy link
Contributor

uBlock-user commented Oct 4, 2021

buyers click on those links to buy products via amazon and blocking anything on that page is blocking functionality for that page.

I don't see any reports of users trying to click on those links(to buy anything if thats even possible, as Amazon doesn't have a "shopping" place like Google does) and failing to do so.

I do understand your opinion, however.

Not an opinion, this is based on reports of users unable to see products ads in order to buy.

Just search for google shopping in the search widget of this tracker and you will see multiple reports of breakage.

@YoshiTabletopGamer
Copy link
Author

YoshiTabletopGamer commented Oct 4, 2021

I know breakage of webpages is a problem, but I was talking about things like this

As you can see, this Amazon search result page has Sponsored results when searching for smartphone on the Amazon website, and that is blocked by ##.AdHolder in EasyList
@uBlock-user

@uBlock-user
Copy link
Contributor

Ask EasyList about this, I don't think they can be compared with Google. Google doesn't sell anything, neither search gets broken on Amazon with that filter unlike on Google Shopping.

@YoshiTabletopGamer
Copy link
Author

So the difficulty is getting a filter that doesn't break things?

@uBlock-user
Copy link
Contributor

#9694
#9600
#9597 (comment)

@YoshiTabletopGamer
Copy link
Author

#9694 #9600 #9597 (comment)

Yeah, the breakage is pretty bad.

@uBlock-user
Copy link
Contributor

Amazon's products and amazon's product ads are distinguishable, unlike Google/Bing shopping pages.

@YoshiTabletopGamer
Copy link
Author

YoshiTabletopGamer commented Nov 13, 2021

Hey, @uBlock-user. I have found out that the Adguard Base filterlist has a working filter for this issue
google.*##div[eid][data-async-context] > .sh-sr__shop-result-group[data-hveid]:has(> div[class] > div[class] > .sh-sp__plain) at https://github.com/AdguardTeam/AdguardFilters/blob/d3bc94a4673d058be55ec3f9a5d59dbfaf137a4b/EnglishFilter/sections/css_extended.txt#L441-L442.

Could you consider its inclusion?

EDIT: see #10132 (comment)

@YoshiTabletopGamer
Copy link
Author

It looks like Adguard added that filter to Adguard Base here AdguardTeam/AdguardFilters@8b60528

@uBlock-user
Copy link
Contributor

Could you consider its inclusion?

Did you test it on the google shopping page ?

@YoshiTabletopGamer
Copy link
Author

Yes, and there's no breakage.

@YoshiTabletopGamer
Copy link
Author

Apparently, that one works for Firefox only?

@YoshiTabletopGamer
Copy link
Author

YoshiTabletopGamer commented Nov 13, 2021

Google must have changed the layout, not working anymore.

@YoshiTabletopGamer
Copy link
Author

The div[eid][data-async-context] part makes it not work, because there's no div with eid attribute anymore.

@uBlock-user
Copy link
Contributor

Let AdGuard team know about it, so they can correct the filter.

@Yuki2718
Copy link
Contributor

Yuki2718 commented Nov 23, 2021

AdguardTeam/AdguardFilters@5caf338
Google certainly changed the layout and I think we can and should fix those "Ads" in shopping pages even though the whole page is ad. Having revisited past issues, on my end these filters work without breakage:

www.google.*##*:not([class^="sh-sr__shop-result-group"]) #kp-wp-tab-overview g-dropdown-menu:upward(#kp-wp-tab-overview > div):has(a[href*="/aclk?"][href$="adurl="])
www.google.*##.sh-sr__shop-result-group[data-hveid] .sh-sp__btn:matches-css(font-size: 13px):upward(.sh-sr__shop-result-group):has(a[href^="/aclk"][href$="adurl="])
www.google.*##c-wiz div[data-hveid] > div > div > ul:has(> li[style="max-width:200px;"] > div > div[data-hveid] > a[href*="/aclk"]):upward(3)

alternating

www.google.*##:not(:matches-path(/^/search\?.*tbm=shop|^/shopping/)) *:not([class^="sh-sr__shop-result-group"]) [href^="/aclk"][href$="adurl="]:upward(3)
www.google.*##[href^="https://www.googleadservices.com/pagead/aclk?"][href$="adurl="]:upward(3)

The div[eid][data-async-context] part makes it not work, because there's no div with eid attribute anymore.

google.*#?#div[eid][data-async-context] > .sh-sr__shop-result-group[data-hveid]:has(> div[class] > div[class] > .sh-sp__plain) by AG is not obsolete, it targets different ads than the new filter added to Base. My filters cover both and one more. matches-path seems not to be mandatory, but we can use it if breakage is reported.

@uBlock-user
Copy link
Contributor

Having revisited past issues, on my end these filters work without breakage:

Okay, add them.

@uBlock-user
Copy link
Contributor

but do not remove

www.google.*##:not(:matches-path(/^/search\?.*tbm=shop|^/shopping/)) a[href*="/aclk?"][data-al]:upward(2)
www.google.*##[href^="https://www.googleadservices.com/pagead/aclk?"][href$="adurl="]:upward(3)

We need to take wait and watch approach here.

@Yuki2718
Copy link
Contributor

It's no urgent. I'm struggling to reproduce www.google.*##c-wiz div[data-hveid] > div > div > ul:has(> li[style="max-width:200px;"] > div > div[data-hveid] > a[href*="/aclk"]):upward(3) - shown on only one Firefox profie when searched for マスク on shopping, but can not reproduce on any other profiles with the exact URL, or even on the same profile with new window. My concern is the tag name c-wiz may not be stable.

Yuki2718 added a commit that referenced this issue Nov 23, 2021
@Yuki2718
Copy link
Contributor

www.google.*##:not(:matches-path(/^/search\?.*tbm=shop|^/shopping/)) a[href*="/aclk?"][data-al]:upward(2) should not be removed, was typo and I meant www.google.*##:not(:matches-path(/^/search\?.*tbm=shop|^/shopping/)) *:not([class^="sh-sr__shop-result-group"]) [href^="/aclk"][href$="adurl="]:upward(3).

@uBlock-user
Copy link
Contributor

uBlock-user commented Nov 23, 2021

I meant for both existing matches-path() filters untill we know for sure, so we don't have to do another back-and-forth for google shopping pages again.

@Yuki2718
Copy link
Contributor

I understand, and added only filters that specifically targets Ads in shopping pages. They should only complement the current filters which targets other pages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants