r69632 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r69631‎ | r69632 | r69633 >
Date:16:18, 20 July 2010
Author:adam
Status:ok
Tags:
Comment:
Fixing indentation on r69481
Modified paths:
  • /trunk/extensions/UsabilityInitiative/UsabilityInitiative.hooks.php (modified) (history)
  • /trunk/extensions/UsabilityInitiative/Vector/Modules/SimpleSearch/SimpleSearch.js (modified) (history)
  • /trunk/extensions/UsabilityInitiative/Vector/Vector.combined.js (modified) (history)
  • /trunk/extensions/UsabilityInitiative/Vector/Vector.hooks.php (modified) (history)

Diff [purge]

Index: trunk/extensions/UsabilityInitiative/Vector/Modules/SimpleSearch/SimpleSearch.js
@@ -63,9 +63,9 @@
6464 .bind( 'submit', function() {
6565 $input.trigger( 'focus' );
6666 } );
67 - if ( $input.val() == '' ) {
68 - $input.trigger( 'blur' );
69 - }
 67+ if ( $input.val() == '' ) {
 68+ $input.trigger( 'blur' );
 69+ }
7070 } );
7171 }
7272 $j( '#searchInput, #searchInput2, #powerSearchText, #searchText' ).suggestions( {
Index: trunk/extensions/UsabilityInitiative/Vector/Vector.hooks.php
@@ -17,13 +17,13 @@
1818 array( 'src' => 'Modules/ExpandableSearch/ExpandableSearch.js', 'version' => 5 ),
1919 array( 'src' => 'Modules/EditWarning/EditWarning.js', 'version' => 10 ),
2020 array( 'src' => 'Modules/FooterCleanup/FooterCleanup.js', 'version' => 5 ),
21 - array( 'src' => 'Modules/SimpleSearch/SimpleSearch.js', 'version' => 24 ),
 21+ array( 'src' => 'Modules/SimpleSearch/SimpleSearch.js', 'version' => 25 ),
2222 ),
2323 'combined' => array(
24 - array( 'src' => 'Vector.combined.js', 'version' => 68 ),
 24+ array( 'src' => 'Vector.combined.js', 'version' => 69 ),
2525 ),
2626 'minified' => array(
27 - array( 'src' => 'Vector.combined.min.js', 'version' => 69 ),
 27+ array( 'src' => 'Vector.combined.min.js', 'version' => 70 ),
2828 ),
2929 );
3030 static $modules = array(
Index: trunk/extensions/UsabilityInitiative/Vector/Vector.combined.js
@@ -573,9 +573,9 @@
574574 .bind( 'submit', function() {
575575 $input.trigger( 'focus' );
576576 } );
577 - if ( $input.val() == '' ) {
578 - $input.trigger( 'blur' );
579 - }
 577+ if ( $input.val() == '' ) {
 578+ $input.trigger( 'blur' );
 579+ }
580580 } );
581581 }
582582 $j( '#searchInput, #searchInput2, #powerSearchText, #searchText' ).suggestions( {
Index: trunk/extensions/UsabilityInitiative/UsabilityInitiative.hooks.php
@@ -86,7 +86,7 @@
8787 array( 'src' => 'js/plugins.combined.js', 'version' => 453 ),
8888 ),
8989 'minified' => array(
90 - array( 'src' => 'js/plugins.combined.min.js', 'version' => 472 ),
 90+ array( 'src' => 'js/plugins.combined.min.js', 'version' => 473 ),
9191 ),
9292 ),
9393 );

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r69481SimpleSearch - moving away from our label-based placeholder text, to input-on...adam17:17, 17 July 2010

Status & tagging log