bug: ABP does not unhide nodes no longer matching :-abp-has(...) procedural cosmetic filters

[krystian3w]

Test links

https://www.e-forma.pl/3575,7nutrition-citrulline-malate-b6-250g-cytrulina.html

http://homeoapteka.pl/2822,bonimed-ziolowy-balsam-kregowy-ojca-grzegorza-100-g.html

Problem:
How can I block only newsletter pop-up when filter:

e-forma.pl,homeoapteka.pl#?##colorbox:-abp-has(#newsletter_colorbox)

breaks too gallery and I cannot see bigger photos products (magnifier/loupe plugin on site is should not enough for all)?

Overlay/background I can detect by style tag and cut only for newsletter:

e-forma.pl,homeoapteka.pl###cboxOverlay:not([style="opacity: 0.9; visibility: visible; display: block; cursor: pointer;"])

Helpful graphics:

​https://i.imgur.com/7s9d5eW.png
​https://i.imgur.com/7s9d5eW.png

Similar bug fixed in uBlock Origin: ​/uBlockOrigin/uBlock-issues/issues/341

[mapx]

Steps:

1. add:

e-forma.pl#?##colorbox:-abp-has(#newsletter_colorbox)
e-forma.pl###cboxOverlay

2. go to https://www.e-forma.pl/3575,7nutrition-citrulline-malate-b6-250g-cytrulina.html
3. The newsletter popup is hidden
4. click the object in the page => the magnifier tool does not work because the filter above (first one) is hiding colorbox div even not containing newsletter_colorbox

ABP should hide only the first 1 colorbox but not the second 1

[krystian3w]

the magnifier tool does not work because the filter above

magnifier tool works, no works gallery after click on picture:

​https://i.imgur.com/T13d679.png

with filter above send.

---

I "found" workaround with (but have to many minuses):

• ​CSS + selector: ###ID:not([style="..."]) + #ID

or

• ​CSS ~ selector: ###ID:not([style="..."]) ~ #ID

minuses:

• this is no works if background code no have specific attrib e.g. :not([style="..."]) and no puted above pop-up html code in dom tree.

• the use of this workaround can further cut mustows of other pop-ups with a identical background and were not a newsletter (save only those with a useful preview of photos in the gallery).

• also closing newsletter pop-up in the past causes the workaround to not work and still break gallery.

[krystian3w]

Another problematic domains with same type newsletter:

oponyalbert.pl
meblezagrosze.pl
annalotan.pl

embeded by colorbox plugin.

[mjethani]

I haven't looked into the specific example posted in the issue description, but I'll just say that element hiding emulation (:-abp-has(), etc.) isn't supposed to unhide anything, so this is sort of "by design". It would be nice if it did work, and we could look into it, but the effort would be non-trivial.

For one, we wouldn't even know which elements we hid because we set style="display: none !mportant" instead of using a style sheet. We've pretty much abandoned our plans to use a style sheet (#6504) because Chromium won't implement tabs.removeCSS (even though I have implemented all the underlying changes, see ​bug #608854).

[mapx]

So, you are saying abp-has is useless => ABP will hide colorbox even if #newsletter_colorbox is not a child of colorbox
(uBo fixed the issue)

[mjethani]

It may be useless right now for the given use case.

For what it's worth, :-abp-has() is "useless" (or unusable) for several other reasons (e.g. #6813). We intend to address these issues next year.

Right now there are only 34 filters in EasyList that use element hiding emulation:

curl https://easylist-downloads.adblockplus.org/easylist.txt | grep '#?#' | wc -l

[krystian3w]

We intend to address these issues next year.

As I did not expect it, they would fix it in three days in uBO addon (beta / dev version).

---

As if the links from the first post down in next year, just try another product available in shops, just as I wrote it has a lot of disadvantages.

---

For one, we wouldn't even know which elements we hid because we set style="display: none !mportant" instead of using a style sheet.

annalotan.pl,e-forma.pl,homeoapteka.pl,oponyalbert.pl,meblezagrosze.pl###cboxOverlay:not([style="opacity: 0.9; visibility: visible; display: block; cursor: pointer;"])
annalotan.pl,e-forma.pl,homeoapteka.pl,oponyalbert.pl,meblezagrosze.pl###cboxOverlay:not([style="opacity: 0.9; visibility: visible; display: block; cursor: pointer;"]) ~ #colorbox

I block overlay pop-up no match with gallery shop ("workaround").

[sebastian]

Sorry, but we switched to GitLab. If this issue is still relevant, please file it again in the ​new issue tracker.

[hfiguiere]

refiled as ​https://gitlab.com/eyeo/adblockplus/adblockpluscore/-/issues/202