Skip to content
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

Fix #871 #872

Closed
Closed

Conversation

yourduskquibbles
Copy link
Contributor

Fixes #871

@gorhill
Copy link
Member

gorhill commented Nov 29, 2017

That is at most an annoyance, there is no list for annoyances on this repo. Please report to Fanboy Annoyance or Adguard Annoyance.

@gorhill gorhill closed this Nov 29, 2017
@yourduskquibbles
Copy link
Contributor Author

ok thanks, will add to my Web Annoyances filter list project.

@gorhill
Copy link
Member

gorhill commented Nov 29, 2017

I see you've put a lot of work in there, and over a long period of time. I am tempted to include it as a stock filter list but it's getting crowded in that section.

@mapx-
Copy link
Contributor

mapx- commented Nov 29, 2017

better adding

100percentfedup.com##.modaal-overlay
@@||100percentfedup.com^$generichide

@yourduskquibbles
Copy link
Contributor Author

@gorhill Wow, that's awesome! My first thought is that if it became a stock filter list, it should NOT be enabled out of the box.

If you were going to add it to available lists, I would probably recommend adding it as an optional (disabled by default) Annoyance list and let people opt-in to it if they want that type of functionality.

If it was enabled by default I think it may piss people off since it alters how sites are visually displayed. I can just imagine it being interpreted poorly as it changes users established workflows and 'desired' site behavior that users have become used to, e.g. The headers taking up part of the screen as you scroll.

There is also a cat-and-mouse game that could develop if it were enabled by default that I've been thinking of opening an issue tracker for on uBO. (i.e. uBO ':style(foo !important;)' filters don't work when the site uses '!important;' on the same 'foo' element in their CSS. I assume due to race condition, but trying to think how uBO filters with '!important;' could have higher priority than the site's elements using the '!important;' tag.

@okiehsch
Copy link
Contributor

okiehsch commented Nov 29, 2017

it should NOT be enabled out of the box

I am pretty sure he does not intend to make an annoyance filter list part of uBO's default setup 😉

Just fyi, your filter 100percentfedup.com##script:inject(abort-on-property-write.js, Math.random)
will break quite a bit of site functionality, for example comments, dropdown menus, embedded twitter videos, etc.
If you want to use a script:injection filter to fix the annoyance, I would suggest
100percentfedup.com##script:inject(abort-current-inline-script.js, setInterval, Blocker)
or simply use the filters suggested by mapx-.

@yourduskquibbles
Copy link
Contributor Author

Thank you @mapx- and @okiehsch! I will use the filters suggested by @mapx- and leave the script inject filters to you guys :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants