| Index: trunk/extensions/ArticleFeedbackv5/modules/ext.articleFeedbackv5/ext.articleFeedbackv5.css |
| — | — | @@ -21,7 +21,6 @@ |
| 22 | 22 | margin: 0; |
| 23 | 23 | right: 0; |
| 24 | 24 | top: 200px; |
| 25 | | - z-index: 10000000; |
| 26 | 25 | } |
| 27 | 26 | .articleFeedbackv5-fixedtabbox { |
| 28 | 27 | position: relative; |
| — | — | @@ -34,9 +33,9 @@ |
| 35 | 34 | margin: 0; |
| 36 | 35 | padding: 5px 10px; |
| 37 | 36 | color: .333; |
| 38 | | - border: 1px solid .555599; |
| 39 | | - border-top: 0px solid .555599; |
| 40 | | - background-color: .eeeef9; |
| | 37 | + border: 1px solid #555599; |
| | 38 | + border-top: 0px solid #555599; |
| | 39 | + background-color: #eeeef9; |
| 41 | 40 | text-align: center; |
| 42 | 41 | writing-mode: tb-rl; |
| 43 | 42 | -moz-transform:rotate(90deg); |
| Index: trunk/extensions/ArticleFeedbackv5/modules/ext.articleFeedbackv5/ext.articleFeedbackv5.js |
| — | — | @@ -111,14 +111,23 @@ |
| 112 | 112 | |
| 113 | 113 | /* Add fixed tab link */ |
| 114 | 114 | if( '3' == linkBucket ) { |
| 115 | | - var $fixedTab = $( '<a href="#mw-articleFeedbackv5" id="articleFeedbackv5-fixedtablink"></a>') |
| | 115 | + var $fixedTab = $( '\ |
| | 116 | + <div id="articleFeedbackv5-fixedtab">\ |
| | 117 | + <div id="articleFeedbackv5-fixedtabbox">\ |
| | 118 | + <a href="#mw-articleFeedbackv5" id="articleFeedbackv5-fixedtablink"></a>\ |
| | 119 | + </div>\ |
| | 120 | + </div>' ); |
| | 121 | + $fixedTab.find( '#articleFeedbackv5-fixedtablink' ) |
| 116 | 122 | .data( 'linkId', 3 ) |
| 117 | 123 | .text( mw.msg( 'articlefeedbackv5-fixedtab-linktext' ) ) |
| 118 | 124 | .click( function( e ) { |
| 119 | 125 | e.preventDefault(); |
| 120 | 126 | clickFeedbackLink( $( e.target ) ); |
| 121 | | - } ) |
| 122 | | - .insertBefore( $aftDiv ); |
| | 127 | + } ); |
| | 128 | + $fixedTab.insertBefore( $aftDiv ); |
| | 129 | + $fixedTab.addClass( 'articleFeedbackv5-fixedtab' ); |
| | 130 | + $fixedTab.find( '#articleFeedbackv5-fixedtabbox' ).addClass( 'articleFeedbackv5-fixedtabbox' ); |
| | 131 | + $fixedTab.find( '#articleFeedbackv5-fixedtablink' ).addClass( 'articleFeedbackv5-fixedtablink' ); |
| 123 | 132 | $aftDiv.articleFeedbackv5( 'addToRemovalQueue', $fixedTab ); |
| 124 | 133 | } |
| 125 | 134 | |
| Index: trunk/extensions/ArticleFeedbackv5/modules/ext.articleFeedbackv5/ext.articleFeedbackv5.startup.js |
| — | — | @@ -45,12 +45,12 @@ |
| 46 | 46 | 'exclude': mw.config.get( 'wgArticleFeedbackv5BlacklistCategories', [] ), |
| 47 | 47 | 'current': mw.config.get( 'wgCategories', [] ) |
| 48 | 48 | }; |
| 49 | | - |
| | 49 | + |
| 50 | 50 | var enable = false; |
| 51 | | - for( cat in categories['current'] ) { |
| | 51 | + for( var cat in categories['current'] ) { |
| 52 | 52 | // Categories are configured with underscores, but article's categories are returned with |
| 53 | 53 | // spaces instead. Revert to underscores here for sane comparison. |
| 54 | | - categories['current'][cat] = categories['current'][cat].replace(/\s/gi, '_'); |
| | 54 | + categories['current'][cat] = categories['current'][cat].replace(/\s/gi, '_'); |
| 55 | 55 | // Check exclusion - exclusion overrides everything else |
| 56 | 56 | if( $.inArray( categories['current'][cat], categories.exclude ) > -1 ) { |
| 57 | 57 | // Blacklist overrides everything else |
| — | — | @@ -61,7 +61,7 @@ |
| 62 | 62 | // categories, and others might be blacklisted - so continue iterating |
| 63 | 63 | enable = true; |
| 64 | 64 | } |
| 65 | | - } |
| | 65 | + } |
| 66 | 66 | |
| 67 | 67 | // Lazy loading |
| 68 | 68 | if ( enable ) { |