bryce/jquery.uls
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove deprecated $.browser

Browse Source 
Fixes issue #33

Change-Id: Idda5d77293baedf8d93b70a146a545ff6986aaa5
This commit is contained in:
Santhosh Thottingal
2012-11-10 17:09:39 +05:30
parent 0be1e38d14
commit 642dd3c8cc
2 changed files with 23 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/jquery.uls.core.js
View File

@@ -215,7 +215,7 @@
// Handle key press events on the menu // Handle key press events on the menu
that.$menu.on('keypress', $.proxy(this.keypress, this) ) that.$menu.on('keypress', $.proxy(this.keypress, this) )
.on('keyup', $.proxy(this.keyup, this) ); .on('keyup', $.proxy(this.keyup, this) );
if ( $.browser.webkit || $.browser.msie ) { if ( this.eventSupported( 'keydown' ) ) {
this.$menu.on( 'keydown', $.proxy( this.keypress, this ) ); this.$menu.on( 'keydown', $.proxy( this.keypress, this ) );
} }
@@ -301,8 +301,17 @@
if ( !this.shown ) { if ( !this.shown ) {
this.show(); this.show();
} }
} },
eventSupported: function ( eventName ) {
var isSupported = eventName in this.$menu;
if ( !isSupported ) {
this.$element.setAttribute( eventName, 'return;' );
isSupported = typeof this.$element[eventName] === 'function';
}
return isSupported;
}
}; };
/* ULS PLUGIN DEFINITION /* ULS PLUGIN DEFINITION

13
src/jquery.uls.languagefilter.js
View File

@@ -50,7 +50,7 @@
listen: function() { listen: function() {
this.$element.on( 'keypress', $.proxy( this.keyup, this ) ) this.$element.on( 'keypress', $.proxy( this.keyup, this ) )
.on( 'keyup', $.proxy( this.keyup, this ) ); .on( 'keyup', $.proxy( this.keyup, this ) );
if ( $.browser.webkit || $.browser.msie ) { if ( this.eventSupported( 'keydown' ) ) {
this.$element.on( 'keydown', $.proxy( this.keyup, this ) ); this.$element.on( 'keydown', $.proxy( this.keyup, this ) );
} }
if ( this.$clear.length ) { if ( this.$clear.length ) {
@@ -250,6 +250,17 @@
matcher.test( $.uls.data.getAutonym( langCode ) ) || matcher.test( $.uls.data.getAutonym( langCode ) ) ||
matcher.test( langCode ) || matcher.test( langCode ) ||
matcher.test( $.uls.data.getScript( langCode ) ); matcher.test( $.uls.data.getScript( langCode ) );
},
eventSupported: function ( eventName ) {
var isSupported = eventName in this.$element;
if ( !isSupported ) {
this.$element.setAttribute( eventName, 'return;' );
isSupported = typeof this.$element[eventName] === 'function';
}
return isSupported;
} }
}; };