-
Notifications
You must be signed in to change notification settings - Fork 723
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
Crashing browser #395
Comments
Hi @PINHOf! Thanks for reporting this. It looks like an issue with the |
Hey @Mottie, I have run a few tests and find out that the problem is on this line: base.$el.bind(o.openOn + base.namespace, function(){ base.focusOn(); }); Which is inside the Switching to: base.el.bind(o.openOn + base.namespace, function(){ base.focusOn(); }); Removes the issue, but I dunno why you were using EDIT 1: Wrong JSFiddle link, this one is OK: https://jsfiddle.net/3cfyjhgw/5/ |
I know the problem is that the |
You're right. The problem is on the function Inside the var start, end, txt, pos,
noFocus = $el.getkeyboard() && $el.getkeyboard().options.noFocus;
if (!noFocus) { $el.focus(); } The variable The solution is just to remove the "!" Check this JSFiddle https://jsfiddle.net/3cfyjhgw/6/ and open the console to inspect. Correct me if I'm wrong, but it seems right. |
Actually, I ended up not using the Try out this updated demo.... note that when |
Hello,
I have a form where I have multiple inputs, so the
accept
button cannot close my keyboard. The form is shown on a modal.I prepared this https://jsfiddle.net/3cfyjhgw/3/ so you can see the problem. Try the following:
accept
clear
accept
Well done, you've now crashed the browser tab. I have tried on Chrome and Firefox and both have the problem.
At first I thought the problem might be due to the
accepted
event I was firing and also thought it was my code problem, but after a few tests I finally realize that the plugin was causing the issue.Thanks.
EDIT 1: I tested again and actually it seems that it only needs to click several times on
accept
button to crash it..The text was updated successfully, but these errors were encountered: