r96573 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r96572‎ | r96573 | r96574 >
Date:15:03, 8 September 2011
Author:catrope
Status:ok
Tags:
Comment:
More style fixes, mostly for r96204
Modified paths:
  • /trunk/extensions/Narayam/js/ext.narayam.core.js (modified) (history)

Diff [purge]

Index: trunk/extensions/Narayam/js/ext.narayam.core.js
@@ -426,7 +426,7 @@
427427 */
428428 this.setup = function() {
429429 // Build the menu
430 - if ( !that.buildMenu() ){
 430+ if ( !that.buildMenu() ) {
431431 // /buildMenu() returned false
432432 // No need to proceed
433433 return;
@@ -448,7 +448,7 @@
449449 else {
450450 $( 'li#pt-narayam' ).addClass( 'narayam-inactive' );
451451 }
452 - // Renew the narayam-enabled cookie. naraym-scheme is renewed by setScheme()
 452+ // Renew the narayam-enabled cookie. narayam-scheme is renewed by setScheme()
453453 if ( enabledCookie ) {
454454 $.cookie( 'narayam-enabled', enabledCookie, {
455455 'path': '/',
@@ -548,10 +548,10 @@
549549 // coming on top of everything.
550550 // TODO: is there a better solution other than hiding it on hover?
551551 if ( $.browser.msie ) {
552 - $( "#narayam-menu" ).hover( function() {
553 - $( "#searchform" ).css( { visibility: "hidden" } );
 552+ $( '#narayam-menu' ).hover( function() {
 553+ $( '#searchform' ).css( 'visibility', 'hidden' );
554554 }, function() {
555 - $( "#searchform" ).css( { visibility: "visible" } );
 555+ $( '#searchform' ).css( 'visibility', 'visible' );
556556 });
557557 }
558558

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r96204workaround for IE bug - activex components like input fields coming on top of...santhosh16:30, 3 September 2011

Status & tagging log