Opened on 01/18/2019 at 11:12:33 AM
Closed on 08/29/2019 at 05:43:18 PM
#7224 closed defect (rejected)
Generic filter text logged if specific filter with same selector matches
Reported by: | greiner | Assignee: | |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | Platform | Keywords: | closed-in-favor-of-gitlab |
Cc: | kzar, sebastian, arthur, mjethani | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Environment
Ubuntu 16.04
Chrome 71
Adblock Plus 3.4.2
How to reproduce
- Add the following filters:
##body example.com##body @@||example.com^$generichide
- Open a new tab and open the Adblock Plus DevTools panel
- Navigate to http://example.com/
Observed behaviour
DevTools panel lists the following filters:
- @@||example.com^$generichide
- ##body
Expected behaviour
DevTools panel lists the following filters:
- @@||example.com^$generichide
- example.com##body
Attachments (0)
Change History (4)
comment:1 Changed on 01/19/2019 at 12:33:36 AM by sebastian
- Cc mjethani added
comment:2 Changed on 01/22/2019 at 12:01:58 PM by greiner
Good idea, preferring domain-specific filters should be sufficient to avoid this issue.
comment:3 Changed on 01/22/2019 at 07:02:21 PM by sebastian
- Priority changed from Unknown to P3
- Ready set
comment:4 Changed on 08/29/2019 at 05:43:18 PM by sebastian
- Keywords closed-in-favor-of-gitlab added
- Resolution set to rejected
- Status changed from new to closed
Sorry, but we switched to GitLab. If this issue is still relevant, please file it again in the new issue tracker.
Note: See
TracTickets for help on using
tickets.
We cannot trace the chain of events from an applied element hiding filter -> injected selector -> blocked element. So the best we can do is looking for which injected selector there is a hidden element on the page, and then find a filter that is active on the given domain and has the same selector. Some edge cases giving inaccurate results might be inevitable. However, maybe a workaround for this particular case could be if we have multiple candidates to prefer domain-specific over generic filters.