Opened on 03/24/2015 at 06:07:59 PM

Closed on 03/25/2015 at 03:35:44 PM

#2212 closed change (fixed)

Intercept clicks on links to subscribe.adblockplus.org on Chrome/Opera/Safari

Reported by: greiner Assignee: sebastian
Priority: P4 Milestone: Adblock-Plus-1.9-for-Chrome-Opera-Safari
Module: Platform Keywords:
Cc: Blocked By:
Blocking: #2208 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/5246523029848064

Description

Background

See #2208

What to change

Make the following changes in the document "click" event handler in include.postload.js:

  • Don't return just because the link doesn't use the "abp:" protocol
  • Intercept links matching /^https?:\/\/subscribe\.adblockplus\.org by adding an additional check similar to the one checking for "abp:subscribe"
  • subscribe.adblockplus.org links should be treated similarly to "abp:subscribe" links (i.e. they should support the same parameters)

What to test

  • Clicking on the "like this one" link at https://adblockplus.org/en/abp_linkfactory should open the options page with filter list title and URL already filled in
  • Clicking on this link should open the options page with filter list title and URL already filled in
  • Clicking on this link or any other regular link should follow the URL that's specified in the link

Attachments (0)

Change History (3)

comment:1 Changed on 03/25/2015 at 02:00:06 PM by sebastian

  • Owner set to sebastian
  • Ready set

comment:2 Changed on 03/25/2015 at 02:02:10 PM by sebastian

  • Priority changed from Unknown to P4
  • Review URL(s) modified (diff)

comment:3 Changed on 03/25/2015 at 03:35:44 PM by sebastian

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
  • Resolution set to fixed
  • Status changed from new to closed

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from sebastian.
 
Note: See TracTickets for help on using tickets.