Opened 2 years ago

Closed 2 years ago

Last modified 19 months ago

#4279 closed defect (fixed)

Whitelisted ad on yahoo.com main page gets a broken link with ABP for Safari iOS

Reported by: scheer Assignee: kzar
Priority: P2 Milestone: Adblock-Plus-for-iOS-next
Module: Adblock-Plus-for-iOS/macOS Keywords: AcceptableAds
Cc: mario, sebastian, kzar Blocked By:
Blocking: Platform: iOS
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29349760/

Description (last modified by scheer)

This issue is very similar to #3050, where a blank page is displayed when the user selects one of the whitelisted ads. The main difference here is this is not from a search result but from Yahoo's front page. The other difference is that Safari opens a new page to load the ad content, as where the ad from Google was attempted to be loaded in-page.

Environment

iPhone 6s Plus
iOS 9.3.1
ABP iOS 1.1.0 (New Filter List as of 1/08/2016 - 09:30)

How to reproduce

  1. With AA enabled, visit www.yahoo.com
  2. Select one of the in-line adverts ($) from the front page.

Observed behaviour

A blank page is opened.

Expected behaviour

A new page is opened with the advertisement URL.

Change History (7)

comment:1 Changed 2 years ago by scheer

  • Description modified (diff)
  • Summary changed from Whitelisted ad on yahoo.com gets a broken link with ABP for Safari iOS to Whitelisted ad on yahoo.com main page gets a broken link with ABP for Safari iOS

comment:2 Changed 2 years ago by mario

  • Keywords AcceptableAds added

comment:3 Changed 2 years ago by kzar

  • Cc mario sebastian kzar added
  • Priority changed from Unknown to P2
  • Ready set
  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:4 Changed 2 years ago by kzar

  • Owner set to kzar

comment:5 Changed 2 years ago by abpbot

A commit referencing this issue has landed:
Issue 4279 - Fix adverts on the Yahoo homepage

comment:6 Changed 2 years ago by kzar

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:7 Changed 19 months ago by mario

  • Milestone set to Adblock-Plus-for-iOS-next

Batch-adjusted milestone for upcoming release of ABP for iOS.

Note: See TracTickets for help on using tickets.