-
Notifications
You must be signed in to change notification settings - Fork 8
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
Don't use JavaScript alert by default to alert about trk requests #11
Comments
May be a less intrusive way can be found (e.g. notification). I would like it that we try hard not to make browsing with Iridium a pain, even when we did not eliminate all the hidden requests yet. |
I commented in #9, but I think it's related to this issue. I don't care if these messages log to the terminal or the JS console, or whatever but an alert is just too obtrusive. I can't use the browser currently until the alerts are removed :( |
We are aware that the pop ups are too invasive to the work flow. Iridium 43.2 will be released soon and should no longer show pop ups during normal operations. See https://github.com/iridium-browser/iridium-browser/wiki/Releases for the progress. |
When such alert is open, the browser is essentially non responsive and kind of "hangs" until it is confirmed. This needs to change asap. |
Thanks, @longsleep |
Tentative patch: http://lists.inai.de/pipermail/iridium/2015-June/000371.html |
We shouldn't show the JavaScript alert by default when a trk request is performed, but only enable it through a commandline option. At least until we got rid of enough trk requests that the browsing experience is not interrupted constantly.
The text was updated successfully, but these errors were encountered: