Skip to content
This repository has been archived by the owner on Apr 17, 2021. It is now read-only.

Better handling of pop-ups #267

Closed
bbinto opened this issue Dec 28, 2017 · 7 comments
Closed

Better handling of pop-ups #267

bbinto opened this issue Dec 28, 2017 · 7 comments
Labels
backlog feature needs investigation needs ux Needs input from UX team P2 Medium impact and/or medium frequency

Comments

@bbinto
Copy link
Contributor

bbinto commented Dec 28, 2017

https://www.amazon.co.uk/review/R2VVS3QHD9KIE9/ref=cm_cr_dp_title?ie=UTF8&ASIN=B078B5YMPD

Terrible on the firestick pop ups everywhere. Nothing at all like the PC version

Another review:
Pop-ups locking you out and force a restart
It's currently pretty useless because of the limited functionality with lots of bugs too such as pop-ups that lock you out and force you to restart. I love Firefox so I'm hoping Mozilla rolls out updates asap and this gets developed into a full browser.

Another review:
https://www.amazon.de/review/R3V1WG0JC0H49Y/ref=cm_cr_dp_title?ie=UTF8&ASIN=B078B5YMPD

@bbinto bbinto changed the title Better handling from pop-ups Better handling of pop-ups Jan 1, 2018
@bbinto bbinto added the P1 High impact and/or high frequency label Jan 1, 2018
@bbinto bbinto added this to the v1.1 milestone Jan 3, 2018
@bbinto bbinto added the size M 3 pts = 2-3 days = 12 - 18 hours label Jan 3, 2018
@bbinto bbinto added the needs ux Needs input from UX team label Jan 8, 2018
@mcomella
Copy link
Contributor

mcomella commented Jan 11, 2018

Sites this affects (will be updated):

  • radio.de (radio.de doesn't work (pop up issues) #262)
  • Default tiles
    • imdb (click a link and be prompted to open in app, banner at bottom; banner hidden in focus, pop-up is not)
    • metacritic (pop-up ad; not present in focus)
    • The Hollywood Reporter (pop-up ad; not present in focus)

Information from a Wikipedia dive:

  • The style of ads we're trying to block are "hover ads" (edit: actually pop-ups may be problems too)
  • "...but it can be blocked with third-party ad blockers such as AdBlock and Adblock Plus, or by using custom style sheets." (via link)
  • Adblock Plus is open source and the main repo is available under GPLv3 – maybe we could look into sharing that (I think it's compatible with MPLv2)

@mcomella
Copy link
Contributor

mcomella commented Jan 11, 2018

@mcomella
Copy link
Contributor

mcomella commented Jan 11, 2018

Some solutions:

  • Use tracking protection (blocks 3rd party scripts, which might block some ads); edit: testing focus above, it seems to improve my small sample of websites – we should revisit this issue after TP lands
  • With GeckoView, allow add-ons like ABP/ublock
  • Learn from existing solutions, e.g. in existing ad blockers (license pending), to share code/implement our own
  • Write our own implementation that heuristically determines hover ads and removes them (this sounds out of scope for this week)
  • Rather than trying to remove them, maybe we could give the user a quick way to close them (e.g. heuristically find the close button, replace pointer with spatial navigation to more easily close) – this could also get us information on pop-ups that we could use to create a blocklist

@mcomella
Copy link
Contributor

Actually, pop-ups may be a problem too – they're usually not a problem in focus but occasionally a website will redirect the main window to a pop-up – maybe users are seeing this too.

@bbinto bbinto modified the milestones: v1.1, v2.0 Jan 12, 2018
@mcomella
Copy link
Contributor

triage: we decided to fix this with tracking protection for 1.1 but we moved this to 2.0 for further work.

@Sdaswani Sdaswani removed the P1 High impact and/or high frequency label Jan 18, 2018
@bbinto bbinto removed this from the v2.0 milestone Jan 23, 2018
@bbinto bbinto added the P2 Medium impact and/or medium frequency label Jan 23, 2018
@bbinto bbinto added this to the Backlog milestone Feb 16, 2018
@bbinto bbinto added the feature label Mar 16, 2018
@mcomella mcomella added needs investigation and removed size M 3 pts = 2-3 days = 12 - 18 hours labels Mar 16, 2018
@mcomella
Copy link
Contributor

is available under GPLv3 – maybe we could look into sharing that

Apparently using GPL code forces our whole app to be GPL so it looks like that's not an option.

@bbinto bbinto added P3 Low impact and frequency P2 Medium impact and/or medium frequency and removed P2 Medium impact and/or medium frequency P3 Low impact and frequency labels Sep 21, 2018
@athomasmoz athomasmoz removed this from the P3 Backlog milestone Nov 22, 2018
@athomasmoz
Copy link

Closing. Will continue tracking market problem

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backlog feature needs investigation needs ux Needs input from UX team P2 Medium impact and/or medium frequency
Projects
None yet
Development

No branches or pull requests

5 participants