http://www.mediawiki.org/wiki/Special:Code/MediaWiki/62142
Revision: 62142 Author: adam Date: 2010-02-08 22:20:03 +0000 (Mon, 08 Feb 2010) Log Message: ----------- Follow up to r62139. Adding a comment and moving from a while, to a for loop Modified Paths: -------------- trunk/extensions/UsabilityInitiative/UsabilityInitiative.hooks.php trunk/extensions/UsabilityInitiative/js/plugins/jquery.wikiEditor.js trunk/extensions/UsabilityInitiative/js/plugins.combined.js trunk/extensions/UsabilityInitiative/js/plugins.combined.min.js Modified: trunk/extensions/UsabilityInitiative/UsabilityInitiative.hooks.php =================================================================== --- trunk/extensions/UsabilityInitiative/UsabilityInitiative.hooks.php 2010-02-08 22:05:32 UTC (rev 62141) +++ trunk/extensions/UsabilityInitiative/UsabilityInitiative.hooks.php 2010-02-08 22:20:03 UTC (rev 62142) @@ -72,7 +72,7 @@ array( 'src' => 'js/plugins/jquery.namespaceSelect.js', 'version' => 1 ), array( 'src' => 'js/plugins/jquery.suggestions.js', 'version' => 7 ), array( 'src' => 'js/plugins/jquery.textSelection.js', 'version' => 27 ), - array( 'src' => 'js/plugins/jquery.wikiEditor.js', 'version' => 109 ), + array( 'src' => 'js/plugins/jquery.wikiEditor.js', 'version' => 110 ), array( 'src' => 'js/plugins/jquery.wikiEditor.highlight.js', 'version' => 29 ), array( 'src' => 'js/plugins/jquery.wikiEditor.toolbar.js', 'version' => 47 ), array( 'src' => 'js/plugins/jquery.wikiEditor.dialogs.js', 'version' => 12 ), @@ -82,10 +82,10 @@ array( 'src' => 'js/plugins/jquery.wikiEditor.publish.js', 'version' => 2 ), ), 'combined' => array( - array( 'src' => 'js/plugins.combined.js', 'version' => 227 ), + array( 'src' => 'js/plugins.combined.js', 'version' => 228 ), ), 'minified' => array( - array( 'src' => 'js/plugins.combined.min.js', 'version' => 227 ), + array( 'src' => 'js/plugins.combined.min.js', 'version' => 228 ), ), ), ); Modified: trunk/extensions/UsabilityInitiative/js/plugins/jquery.wikiEditor.js =================================================================== --- trunk/extensions/UsabilityInitiative/js/plugins/jquery.wikiEditor.js 2010-02-08 22:05:32 UTC (rev 62141) +++ trunk/extensions/UsabilityInitiative/js/plugins/jquery.wikiEditor.js 2010-02-08 22:20:03 UTC (rev 62142) @@ -301,14 +301,18 @@ return false; } break; + // Intercept all tab events to provide consisten behavior across browsers + // Webkit browsers insert tab characters by default into the iframe rather than changing input focus case 9: var $tabindexList = $j( '[tabindex]:visible' ).sort( function( a, b ) { return a.tabIndex > b.tabIndex ? 1 : -1; } ); - while( ! $tabindexList.eq( 0 ).is( '#' + context.$iframe.attr( 'id' ) ) ) { - $tabindexList = $tabindexList.slice( 1 ); + for( var i=0; i < $tabindexList.length; i++ ) { + if( $tabindexList.eq( i ).is( '#' + context.$iframe.attr( 'id' ) ) ) { + $tabindexList.get( i + 1 ).focus(); + break; + } } - $tabindexList.get( 1 ).focus(); return false; break; } Modified: trunk/extensions/UsabilityInitiative/js/plugins.combined.js =================================================================== --- trunk/extensions/UsabilityInitiative/js/plugins.combined.js 2010-02-08 22:05:32 UTC (rev 62141) +++ trunk/extensions/UsabilityInitiative/js/plugins.combined.js 2010-02-08 22:20:03 UTC (rev 62142) @@ -6734,14 +6734,18 @@ return false; } break; + // Intercept all tab events to provide consisten behavior across browsers + // Webkit browsers insert tab characters by default into the iframe rather than changing input focus case 9: var $tabindexList = $j( '[tabindex]:visible' ).sort( function( a, b ) { return a.tabIndex > b.tabIndex ? 1 : -1; } ); - while( ! $tabindexList.eq( 0 ).is( '#' + context.$iframe.attr( 'id' ) ) ) { - $tabindexList = $tabindexList.slice( 1 ); + for( var i=0; i < $tabindexList.length; i++ ) { + if( $tabindexList.eq( i ).is( '#' + context.$iframe.attr( 'id' ) ) ) { + $tabindexList.get( i + 1 ).focus(); + break; + } } - $tabindexList.get( 1 ).focus(); return false; break; } Modified: trunk/extensions/UsabilityInitiative/js/plugins.combined.min.js =================================================================== --- trunk/extensions/UsabilityInitiative/js/plugins.combined.min.js 2010-02-08 22:05:32 UTC (rev 62141) +++ trunk/extensions/UsabilityInitiative/js/plugins.combined.min.js 2010-02-08 22:20:03 UTC (rev 62142) @@ -447,8 +447,8 @@ if('fn'in $.wikiEditor.modules[module]&&'create'in $.wikiEditor.modules[module].fn){context.modules[module]={};$.wikiEditor.modules[module].fn.create(context,modules[module]);}}}}};context.evt={'keydown':function(event){switch(event.which){case 90:if((event.ctrlKey||event.metaKey)&&context.history.length){if(event.shiftKey){context.historyPosition++;}else{context.historyPosition--;} if(context.history.length+context.historyPosition>=0&&context.historyPosition<0){context.$content.html(context.history[context.history.length+context.historyPosition].html);}else{context.historyPosition=Math.max(-context.history.length,Math.min(context.historyPosition,-1));} return false;} -break;case 9:var $tabindexList=$j('[tabindex]:visible').sort(function(a,b){return a.tabIndex>b.tabIndex?1:-1;});while(!$tabindexList.eq(0).is('#'+context.$iframe.attr('id'))){$tabindexList=$tabindexList.slice(1);} -$tabindexList.get(1).focus();return false;break;} +break;case 9:var $tabindexList=$j('[tabindex]:visible').sort(function(a,b){return a.tabIndex>b.tabIndex?1:-1;});for(var i=0;i<$tabindexList.length;i++){if($tabindexList.eq(i).is('#'+context.$iframe.attr('id'))){$tabindexList.get(i+1).focus();break;}} +return false;break;} return true;},'change':function(event){event.data.scope='division';var newHTML=context.$content.html();if(context.oldHTML!=newHTML){context.fn.purgeOffsets();context.oldHTML=newHTML;event.data.scope='realchange';} switch(event.which){case 8:break;} return true;},'delayedChange':function(event){event.data.scope='division';var newHTML=context.$content.html();if(context.oldDelayedHTML!=newHTML){context.fn.purgeOffsets();context.oldDelayedHTML=newHTML;event.data.scope='realchange';if(newHTML!==context.history[context.history.length+context.historyPosition].html){context.historyPosition=-1;} _______________________________________________ MediaWiki-CVS mailing list MediaWiki-CVS@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs