-
Notifications
You must be signed in to change notification settings - Fork 80
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
::before
/ ::after
no longer hide node as generic filter?
#464
Comments
@gwarser Did this work before ? |
My Filters shows |
I can confirm first-hand that this has been supported since at least July last year, potentially earlier as well. In fact, DandelionSprout/adfilt#7 was in its modest beginnings about an issue that was resolved by using a |
https://github.com/DandelionSprout/adfilt/blob/master/NorwegianList.txt#L967 through L981 contains several examples of this being applied in regular use, and |
March 2017? BTW; |
The filter requires at least one hostname, it can't be used without a hostname. This has been like this since at least Dec 2017, I didn't check earlier. Edit: This has been like this since gorhill/uBlock#2448 was fixed, it is not true that this is a new behavior. |
If this was the case, then shouldn't uBO Legacy fail to load such generic rules too? |
I would expect so. I will have to investigate further then. |
Ok I was wrong: it seems this is a regression from gorhill/uBlock@261ef8c. Meanwhile the workaround is to make the filter specific (which is always the preferred approach anyways). |
Related issue: - uBlockOrigin/uBlock-issues#464 Regression from: 261ef8c?diff=unified#diff-3b15596213ed9ba37fb5b8bb1402a6c2R599 Pseudoclass-based generic cosmetic filters were improperly seen as invalid following the regression.
This comment has been minimized.
This comment has been minimized.
Try found article from Google, also clear cookies for medonet. |
These are tests for CSS pseudo elements, they succeed: https://gorhill.github.io/uBlock/tests/css-selector-based-cosmetic-filters.html. |
OK - I suppose similar glitch AS generic:
When for my filter does not work but @gwarser have sometime blank page (breakage). |
Those filters are specific, they are injected unconditionally so they can't suffer from issues affecting generic filters. |
@krystian3w it's about this case where targeted class is added dynamically by script and because of this uBO sometimes does not notice - gorhill/uBlock#1885 ? |
Yes. |
Related issues: - uBlockOrigin/uBlock-issues#464 - gorhill#290 Regression from: 4e91155 Pseudoclass-based generic cosmetic filters were improperly seen as invalid following the regression. Co-authored-by: gorhill <585534+gorhill@users.noreply.github.com>
:before
/ :after
no longer hide node as generic filter?
:before
/ :after
no longer hide node as generic filter?::before
/ ::after
no longer hide node as generic filter?
Prerequisites
Description
##div[class="widget polarisSpid"]::before
no longer cut / hide pseudoelements as generic cosmetic filter.A specific URL where the issue occurs
DandelionSprout/adfilt#7 (comment)
Steps to Reproduce
paste to my filters panel:
open demo site
blurred on text is still visible:
Expected behavior:
recover works filter as generic
Actual behavior:
filter need TLD or domain name to works
Your environment
no affected versions:
uBO <= 1.17.4
The text was updated successfully, but these errors were encountered: