Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

account for retooled key events handling in newer Opera #31

Merged
merged 1 commit into from

2 participants

@miketaylr

This will ensure that this plugin continues to work once key event changes land in Opera Next. Thanks!

@miketaylr

anyone home? :)

@agarzola agarzola merged commit fa78892 into agarzola:master
@agarzola
Owner

I realize it took forever for me to merge this in. Apologies!

I appreciate your patience and your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 2, 2012
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 jquery.autocomplete.js
View
4 jquery.autocomplete.js
@@ -94,8 +94,8 @@ $.Autocompleter = function(input, options) {
}
});
- // only opera doesn't trigger keydown multiple times while pressed, others don't work with keypress at all
- $input.bind(($.browser.opera ? "keypress" : "keydown") + ".autocomplete", function(event) {
+ // older versions of opera don't trigger keydown multiple times while pressed, others don't work with keypress at all
+ $input.bind(($.browser.opera && !'KeyboardEvent' in window) ? "keypress" : "keydown") + ".autocomplete", function(event) {
// a keypress means the input has focus
// avoids issue where input had focus before the autocomplete was applied
hasFocus = 1;
Something went wrong with that request. Please try again.