r89805 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r89804‎ | r89805 | r89806 >
Date:21:59, 9 June 2011
Author:foxtrott
Status:deferred
Tags:
Comment:
some cleanup
Modified paths:
  • /trunk/extensions/Lingo/libs/Lingo.js (modified) (history)

Diff [purge]

Index: trunk/extensions/Lingo/libs/Lingo.js
@@ -1,10 +1,6 @@
22 jQuery(function ($){
33
4 - var tooltips = $(".tooltip");
5 - // var tooltip_wrapper = tooltip.find(".tooltip_tipwrapper");
6 - // var tooltip_tip = tooltip_wrapper.find(".tooltip_tip");
7 -
8 - tooltips
 4+ $(".tooltip")
95 .mouseenter(function(){
106
117 var tip = $(this);
@@ -19,7 +15,6 @@
2016
2117 wrapper
2218 .css({
23 - // 'visibility': 'hidden',
2419 'visibility': 'visible',
2520 'display': 'block',
2621 'width': '10000000px'
@@ -47,12 +42,7 @@
4843
4944 tipdef.width( maxAvailableWidth );
5045
51 - // tipdef.css({
52 - // 'position': 'fixed',
53 - // 'top': '0px',
54 - // 'left': '0px'
55 - // });
56 - // if constrained to the window width, i.e.
 46+ // height if constrained to the window width, i.e.
5747 // the minimum width necessary if the full window width were available
5848 var heightAtMaxWidth = tipdef.height();
5949
@@ -85,14 +75,14 @@
8676
8777 // if it is too large anyway, just set max available width and be
8878 // done with it
89 - wrapper.css({
90 - 'width': maxAvailableWidth + 'px',
91 - 'padding-left': '10px',
92 - 'left': '0px',
93 - 'top': '0px',
94 - 'padding-bottom': '0px',
95 - 'padding-top' : (tip.outerHeight() + 5 ) +'px'
96 - });
 79+ wrapper.css({
 80+ 'width': maxAvailableWidth + 'px',
 81+ 'padding-left': '10px',
 82+ 'left': '0px',
 83+ 'top': '0px',
 84+ 'padding-bottom': '0px',
 85+ 'padding-top' : (tip.outerHeight() + 5 ) +'px'
 86+ });
9787
9888 } else {
9989
@@ -143,7 +133,7 @@
144134 if ( placeAbove ) {
145135 wrapper.css({
146136 'top': ( - tipdef.outerHeight() - 5) + 'px',
147 - 'padding-bottom': '10px',
 137+ 'padding-bottom': (tip.outerHeight() + 5 ) +'px',
148138 'padding-top' : '0px'
149139 });
150140
@@ -166,7 +156,6 @@
167157
168158 wrapper
169159 .css({
170 - // 'width': '100%',
171160 'height': 'auto',
172161 'visibility': 'visible',
173162 'display': 'none'

Status & tagging log