Opened on 02/06/2016 at 01:07:26 PM

Closed on 02/10/2016 at 05:47:25 PM

Last modified on 03/08/2016 at 07:32:43 AM

#3636 closed defect (fixed)

"Block element" dialog hides element even if added filters don't apply due to $genricblock

Reported by: sebastian Assignee: sebastian
Priority: P3 Milestone: Adblock-Plus-1.11-for-Chrome-Opera-Safari
Module: Platform Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

Description (last modified by sebastian)

How to reproduce

  1. Add filter @@||^$genericblock
  2. Go to
  3. Click ABP toolbar icon
  4. Choose "Block element"
  5. Click on the Google logo
  6. Remove the $domain option from the generated filters
  7. Add filters
  8. Reload the page

Observed behaviour

When adding the filters, the the Google logo is instantly removed. When reloading the page however, the logo shows up again. That is because the logic that checks whether the element is supposed to be blocked, when adding the filters, doesn't consider the $genericblock exception rule added earlier, and therefore hides the element, even though the new filters don't actually match.

In some edge cases, this might also cause elements to be hidden that aren't actually blocked, when loading documents that are whitelisted by $genericblock, since the same logic is used for the element collapsing. This issue is the equivalent of #3170 for the $genericblock option.

Expected behaviour

The elements should not be hidden when a $genericblock exception rule for the document whitelists the added filters.

Attachments (0)

Change History (4)

comment:1 Changed on 02/06/2016 at 01:20:09 PM by sebastian

  • Description modified (diff)

comment:2 Changed on 02/06/2016 at 01:21:08 PM by sebastian

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed on 02/10/2016 at 05:47:25 PM by sebastian

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

comment:4 Changed on 03/03/2016 at 10:58:25 AM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Fixed. Couldn't reproduce the issue with the steps and tried it out on both Google and the BBC logo.

Chrome 35 / 47 / Windows 7 x86
Chrome 48 / Ubuntu 14.04 x64
Opera 25 / 35 / Windows 7 x86
Opera 35 / Ubuntu 14.04 x64
Safari 6 (OS X 10.8) / Safari 8 (OS X 10.10) / Safari 9 (OS X 10.11)

Last edited on 03/08/2016 at 07:32:43 AM by Ross

Add Comment

Modify Ticket

Change Properties
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.