You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lo...@apache.org on 2011/11/29 13:58:36 UTC

svn commit: r1207854 - in /myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script: tobago-menu.js tobago-sheet.js tobago.js

Author: lofwyr
Date: Tue Nov 29 12:58:35 2011
New Revision: 1207854

URL: http://svn.apache.org/viewvc?rev=1207854&view=rev
Log:
code style: using parseInt(s) instead of parseInt(s, 10)

Modified:
    myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-menu.js
    myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-sheet.js
    myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js

Modified: myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-menu.js
URL: http://svn.apache.org/viewvc/myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-menu.js?rev=1207854&r1=1207853&r2=1207854&view=diff
==============================================================================
--- myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-menu.js (original)
+++ myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-menu.js Tue Nov 29 12:58:35 2011
@@ -155,7 +155,7 @@ Tobago.Menu.open = function(event) {
     sub.css('visibility', 'visible');
     
     // IE6 select-tag fix
-    if (jQuery.browser.msie && parseInt(jQuery.browser.version, 10) <= 6) {
+    if (jQuery.browser.msie && parseInt(jQuery.browser.version) <= 6) {
       //          sub.css('width', sub.width());
       //          sub.css('height', sub.height());
       //          sub.css('display', 'none');
@@ -163,8 +163,8 @@ Tobago.Menu.open = function(event) {
       var iframe = sub.children("iframe:first");
       iframe.css('width', sub.outerWidth());
       iframe.css('height', sub.outerHeight());
-      iframe.css('left', -(parseInt(sub.css('border-left-width'), 10) + parseInt(sub.css('padding-left'), 10)));
-      iframe.css('top', -(parseInt(sub.css('border-top-width'), 10) + parseInt(sub.css('padding-top'), 10)));
+      iframe.css('left', -(parseInt(sub.css('border-left-width')) + parseInt(sub.css('padding-left'))));
+      iframe.css('top', -(parseInt(sub.css('border-top-width')) + parseInt(sub.css('padding-top'))));
     }
   }
       
@@ -250,7 +250,7 @@ Tobago.Menu.init = function(elements) {
   // IE6 select-tag fix
   // put a iframe inside the div, so that a <select> tag doesn't shine through.
   // the iframe must be resized (see above)
-  if (jQuery.browser.msie && parseInt(jQuery.browser.version, 10) <= 6) {
+  if (jQuery.browser.msie && parseInt(jQuery.browser.version) <= 6) {
     menus.children("ol").prepend(
         "<iframe class='tobago-menu-ie6bugfix' src='" + Tobago.blankPage + "'></iframe>");
   }

Modified: myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-sheet.js
URL: http://svn.apache.org/viewvc/myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-sheet.js?rev=1207854&r1=1207853&r2=1207854&view=diff
==============================================================================
--- myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-sheet.js (original)
+++ myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago-sheet.js Tue Nov 29 12:58:35 2011
@@ -300,7 +300,7 @@ Tobago.Sheet.prototype.setup = function(
   //      this.adjustScrollBars();
   this.setScrollPosition();
 
-  this.firstRowIndex = parseInt(this.getRows().eq(0).attr("rowIndexInModel"), 10);
+  this.firstRowIndex = parseInt(this.getRows().eq(0).attr("rowIndexInModel"));
   this.rowCount = this.getRows().size();
 
   if (this.selectable
@@ -730,7 +730,7 @@ Tobago.Sheet.prototype.endResize = funct
         Tobago.Sheet.fixIE67ColWidth(col);
       }
 
-      var index = parseInt(columnNr, 10) + 1;
+      var index = parseInt(columnNr) + 1;
       var tds = jQuery("td:nth-child(" + index + ")", table);
       var horizontalDiff = 0;
       var widthWithoutPadding = width;

Modified: myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js
URL: http://svn.apache.org/viewvc/myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js?rev=1207854&r1=1207853&r2=1207854&view=diff
==============================================================================
--- myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js (original)
+++ myfaces/tobago/trunk/tobago-theme/tobago-theme-standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js Tue Nov 29 12:58:35 2011
@@ -885,7 +885,7 @@ var Tobago = {
       shield.attr('id', maxModalPopup.attr('id') + '::shield');
 
       // IE6 doesn't support position:fixed
-      if (jQuery.browser.msie && parseInt(jQuery.browser.version, 10) <= 6) {
+      if (jQuery.browser.msie && parseInt(jQuery.browser.version) <= 6) {
         shield.css({
           position: 'absolute',
           left: -maxModalPopup.offset().left,