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

I'm having problems with the Subscribe buttons #438

Closed
DandelionSprout opened this issue Sep 5, 2018 · 9 comments
Closed

I'm having problems with the Subscribe buttons #438

DandelionSprout opened this issue Sep 5, 2018 · 9 comments
Labels
bug something isn't working feedback wanted provide your input web front-end user interface

Comments

@DandelionSprout
Copy link
Contributor

DandelionSprout commented Sep 5, 2018

In the past 24 hours, nothing seems to happen (i.e. no "Do you want to add this list?" message) when I click on a list's Subscribe button on FilterLists.com, and I can't seem to identify the reason for it.

The subscription links on other sites (e.g. https://adblockplus.org/subscriptions) seem to work perfectly well, so can anyone confirm that they're having problems with this as well?

My setup:
Software: Nano Adblocker 1.0.0.66 (Chromium version)
Browser: Google Chrome 69.0.3497.81 64-bit stable
OS: Windows 10 Pro 10.0.17134 64-bit

@hawkeye116477
Copy link
Contributor

hawkeye116477 commented Sep 5, 2018

I can also confirm that subscribe links don't work.

@DandelionSprout
Copy link
Contributor Author

The three most likely possibilities that I can semi-qualifiedly guess on, are:

  1. Problems with the button scripting.
  2. Problems with Unicode in the links.
  3. That someone (possibly ABP) suddenly thinks that FilterLists.com is an intrusive site, and has blacklisted the abp:// protocol from working on there.

I can't confirm any of those three possibilities myself, however. I guess Collin (who lives in the US and is therefore asleep right now) could need to do some investigation on this matter when he wakes up.

@collinbarrett collinbarrett added bug something isn't working web front-end user interface high priority must fix ASAP labels Sep 5, 2018
@collinbarrett
Copy link
Owner

collinbarrett commented Sep 5, 2018

Ok, I think I know what is going on. @gorhill has said before that the script that uBlock Origin (and I guess Nano as well? I haven't used that fork at all.) uses to find the links is short-lived. If it doesn't find any subscribe links in the first few seconds, it bails.

Since I removed the Subscribe links from the main grid a couple of days ago (to preserve real estate), my guess is this script bails out since it doesn't see a subscribe link quickly. I can confirm that by refreshing the page, and then clicking a "Details"->"Subscribe" very quickly. That seems to work on my end with uBO in Firefox. But, if I wait a couple seconds before clicking a "Details", the links don't work.

I think the solution will be to add at least one abp: link to the page ASAP so that the uBO/Nano script will hang around. I might try adding this to the raw HTML (outside of React), so as to be served ASAP, and just hide it off in a corner somewhere.

Will look into it.

@gorhill
Copy link

gorhill commented Sep 5, 2018

@collinbarrett There is a special detection case for https://filterlists.com in uBO: the subscribe scriptlet will stick around if it finds a <link rel="canonical" href="https://filterlists.com/"> in the DOM. So just adding this should be enough.

collinbarrett added a commit that referenced this issue Sep 5, 2018
@collinbarrett
Copy link
Owner

Awesome, thanks. Trying that out now.

@collinbarrett
Copy link
Owner

Nice. That seems to have resolved it for both uBO and its fork Nano in Chrome and Firefox on Windows.

@DandelionSprout and @hawkeye116477 , please confirm.

@collinbarrett collinbarrett added the feedback wanted provide your input label Sep 5, 2018
@DandelionSprout
Copy link
Contributor Author

Seems to work on my end now, so I guess I can confirm.

@hawkeye116477
Copy link
Contributor

Yes, works perfectly 😄.

@collinbarrett
Copy link
Owner

Great, thanks for the feedback.

hawkeye116477 pushed a commit to hawkeye116477/FilterLists that referenced this issue Feb 4, 2019
@collinbarrett collinbarrett removed the high priority must fix ASAP label Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something isn't working feedback wanted provide your input web front-end user interface
Projects
None yet
Development

No branches or pull requests

4 participants