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

Vimium disables/breaks confirm and alert on all pages #2082

Open
nickstenning opened this issue Apr 5, 2016 · 3 comments
Open

Vimium disables/breaks confirm and alert on all pages #2082

nickstenning opened this issue Apr 5, 2016 · 3 comments

Comments

@nickstenning
Copy link

I recently noticed that various pages weren't working for me as they should. I eventually tracked this down to window.confirm always returning false.

Steps to reproduce

  1. Create a new Chrome browser profile
  2. Install Vimium
  3. Open any web page and pop open the developer console
  4. Execute confirm("check confirm?");
  5. Execute alert("hello world!");

Expected behaviour

Steps 4. and 5. should both result in browser popups. confirm should return true or false depending on whether the user clicks 'OK' or 'Cancel' on the popup.

Observed behaviour

confirm(...) returns false immediately, without displaying a popup. alert similarly returns immediately, without displaying a popup.

Disabling Vimium doesn't immediately fix the issue, but a subsequent browser restart will. Reenabling Vimium causes the issue to recur.

Version details

Mac OS X 10.11.3 (15D21)
Chrome 51.0.2693.2 dev
Vimium 1.54

@smblott-github
Copy link
Collaborator

I recently noticed that various pages weren't working for me as they should.

In what way weren't pages working, @nickstenning? Using Vimium, you should rarely encounter confirm().

@nickstenning
Copy link
Author

I'm not sure I understand your question. If a page has a confirm() action on a button, Vimium shouldn't make it impossible to click that button, and previously (i.e. only a few days ago) it didn't.

@smblott-github
Copy link
Collaborator

I cannot reproduce this, @nickstenning. I've tried chrome 49, 50 and 51.

snapshot

Also, the word confirm is only used in one place in the entire Vimium code base...

$ grep -w confirm **/*.coffee
background_scripts/main.coffee:      runCommand = confirm """

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

No branches or pull requests

2 participants