| Index: trunk/extensions/Narayam/js/ext.narayam.core.js |
| — | — | @@ -505,12 +505,12 @@ |
| 506 | 506 | var $input = $( '<input type="radio" name="narayam-input-method" class="narayam-scheme" />' ); |
| 507 | 507 | $input.attr( 'id', 'narayam-' + scheme ).val( scheme ); |
| 508 | 508 | |
| 509 | | - var $narayamMenuItemLabel = $( '<label />' ) |
| | 509 | + var $narayamMenuItemLabel = $( '<label>' ) |
| 510 | 510 | .attr( 'for' ,'narayam-' + scheme ) |
| 511 | 511 | .append( $input ) |
| 512 | 512 | .append( mw.message( "narayam-"+ scheme ).escaped() ); |
| 513 | 513 | |
| 514 | | - var $narayamMenuItem = $( '<li/>' ) |
| | 514 | + var $narayamMenuItem = $( '<li>' ) |
| 515 | 515 | .append( $input ) |
| 516 | 516 | .append( $narayamMenuItemLabel ); |
| 517 | 517 | return $narayamMenuItem; |
| — | — | @@ -523,7 +523,7 @@ |
| 524 | 524 | this.buildMenuItems = function(){ |
| 525 | 525 | var haveSchemes = false; |
| 526 | 526 | // Build schemes option list |
| 527 | | - var $narayamMenuItems = $( '<ul/>' ); |
| | 527 | + var $narayamMenuItems = $( '<ul>' ); |
| 528 | 528 | var count = 1; |
| 529 | 529 | var seen = []; |
| 530 | 530 | |
| — | — | @@ -572,15 +572,15 @@ |
| 573 | 573 | .attr( 'title', mw.msg( 'narayam-checkbox-tooltip' ) ) |
| 574 | 574 | .click( that.toggle ); |
| 575 | 575 | |
| 576 | | - var $label = $( '<label for="narayam-toggle" />' ); |
| | 576 | + var $label = $( '<label>' ).attr( 'for', 'narayam-toggle' ); |
| 577 | 577 | $label |
| 578 | 578 | .text( mw.msg( 'narayam-toggle-ime', shortcutText() ) ) |
| 579 | 579 | .prepend( $checkbox ) |
| 580 | | - .attr( 'title', mw.msg( 'narayam-checkbox-tooltip' ) ); |
| | 580 | + .prop( 'title', mw.msg( 'narayam-checkbox-tooltip' ) ); |
| 581 | 581 | |
| 582 | | - $narayamMenuItems.append( $( '<li class="narayam-more-imes-link" />') |
| | 582 | + $narayamMenuItems.append( $( '<li>' ).addClass( 'narayam-more-imes-link' ) |
| 583 | 583 | .append( |
| 584 | | - $( '<a/>' ) |
| | 584 | + $( '<a>' ) |
| 585 | 585 | .text( mw.msg( 'narayam-more-imes' ) ) |
| 586 | 586 | .click( function() { |
| 587 | 587 | $('.narayam-scheme-dynamic-item').toggle('fast'); |
| — | — | @@ -619,23 +619,18 @@ |
| 620 | 620 | |
| 621 | 621 | var helppage = mw.config.get( 'wgNarayamHelpPage' ); |
| 622 | 622 | if ( helppage ) { |
| 623 | | - $narayamMenuItems.append( $( '<li class="narayam-help-link" />') |
| 624 | | - .append( |
| 625 | | - $( '<a/>' ) |
| 626 | | - .text( mw.msg( 'narayam-help' ) ) |
| 627 | | - .attr( |
| 628 | | - 'href', |
| 629 | | - mw.util.wikiGetlink( helppage ) |
| 630 | | - ) |
| 631 | | - ) |
| 632 | | - ); |
| | 623 | + var $link = $( '<a>' ) |
| | 624 | + .text( mw.msg( 'narayam-help' ) ) |
| | 625 | + .prop( 'href', mw.util.wikiGetlink( helppage ) ); |
| | 626 | + var $li = $( '<li>' ).addClass( 'narayam-help-link' ); |
| | 627 | + $narayamMenuItems.append( $li.append( $link ) ); |
| 633 | 628 | } |
| 634 | 629 | |
| 635 | | - $narayamMenuItems.prepend( $( '<li/>' ).append( $label ) ); |
| 636 | | - var $menuItemsDiv = $( '<div id="narayam-menu-items" class="menu-items" />' ); |
| 637 | | - $menuItemsDiv |
| | 630 | + $narayamMenuItems.prepend( $( '<li>' ).append( $label ) ); |
| | 631 | + return $( '<div>' ) |
| | 632 | + .attr( 'id', 'narayam-menu-items' ) |
| | 633 | + .addClass( 'menu-items' ) |
| 638 | 634 | .append( $narayamMenuItems ); |
| 639 | | - return $menuItemsDiv; |
| 640 | 635 | } |
| 641 | 636 | |
| 642 | 637 | |
| — | — | @@ -644,18 +639,17 @@ |
| 645 | 640 | */ |
| 646 | 641 | this.buildMenu = function() { |
| 647 | 642 | var $menuItemsDiv = that.buildMenuItems(); |
| 648 | | - var $menu = $( '<div id="narayam-menu" class="narayam-menu" />'); |
| 649 | | - $menu |
| 650 | | - .append( |
| 651 | | - $( '<a href="#" />' ) |
| 652 | | - .text( mw.msg( 'narayam-menu' ) ) |
| 653 | | - .attr( 'title', mw.msg( 'narayam-menu-tooltip' ) ) |
| 654 | | - ) |
| 655 | | - .append( $menuItemsDiv ); |
| | 643 | + var $menu = $( '<div>' ) |
| | 644 | + .attr( 'id', 'narayam-menu' ) |
| | 645 | + .addClass( 'narayam-menu' ); |
| | 646 | + var $link = $( '<a>' ) |
| | 647 | + .prop( 'href', '#' ) |
| | 648 | + .text( mw.msg( 'narayam-menu' ) ) |
| | 649 | + .attr( 'title', mw.msg( 'narayam-menu-tooltip' ) ); |
| | 650 | + |
| | 651 | + $menu.append( $link ).append( $menuItemsDiv ); |
| | 652 | + var $li = $( '<li>' ).attr( 'id', 'pt-narayam' ).append( $menu ); |
| 656 | 653 | |
| 657 | | - var $li = $( '<li id="pt-narayam" />' ); |
| 658 | | - $li.append( $menu ); |
| 659 | | - |
| 660 | 654 | // If rtl, add to the right of top personal links. Else, to the left |
| 661 | 655 | var fn = $( 'body' ).hasClass( 'rtl' ) ? "append" : "prepend"; |
| 662 | 656 | $( '#p-personal ul:first' )[fn]( $li ); |