Merge pull request niklasvh#100 from BugHerd/linting

Added a sublime-project file for Sublime Text 2 with linting and standards.

Author: niklasvh

.gitignore

@@ -10,3 +10,4 @@ image.jpg
 /.project
 /.settings/
 .envrc
+*.sublime-workspace

html2canvas.sublime-project

@@ -0,0 +1,57 @@
+{
+	"folders":
+	[
+		{
+			"path": ".",
+			"folder_exclude_patterns": [".svn", ".git", ".hg", "CVS", "node_modules"],
+			"file_exclude_patterns": ["*.sublime-workspace"]
+		}
+	],
+	"settings":
+	{
+		// The number of spaces a tab is considered equal to
+		"tab_size": 4,
+
+		// Set to true to insert spaces when tab is pressed
+		"translate_tabs_to_spaces": true,
+
+		// If translate_tabs_to_spaces is true, use_tab_stops will make tab and
+		// backspace insert/delete up to the next tabstop
+		"use_tab_stops": false,
+
+		// Set to false to disable detection of tabs vs. spaces on load
+		"detect_indentation": false,
+
+		// Set to true to removing trailing white space on save
+		"trim_trailing_white_space_on_save": true,
+
+		// Set to true to ensure the last line of the file ends in a newline
+		// character when saving
+		"ensure_newline_at_eof_on_save": false,
+
+		// Linting
+		"jshint_options": {
+			"eqeqeq": false,
+
+			"laxbreak": true,
+			"undef": true,
+			"newcap": true,
+			"noarg": true,
+			"strict": false,
+			"trailing": true,
+			"onecase": true,
+
+			"boss": true,
+			"eqnull": true,
+
+			"onevar": false,
+
+			"evil": true,
+			"regexdash": true,
+			"browser": true,
+			"wsh": true,
+			"trailing": true,
+			"sub": true
+		}
+	}
+}

src/Core.js

@@ -20,62 +20,62 @@ function h2clog(a) {
 _html2canvas.Util = {};
 
 _html2canvas.Util.backgroundImage = function (src) {
-  
+
     if (/data:image\/.*;base64,/i.test( src ) || /^(-webkit|-moz|linear-gradient|-o-)/.test( src )) {
         return src;
     }
-    
+
     if (src.toLowerCase().substr( 0, 5 ) === 'url(http://github.com/"'%3C/span%3E%3Cspan%20class=pl-kos%3E) {
         src = src.substr( 5 );
-        src = src.substr( 0, src.length - 2 );                 
+        src = src.substr( 0, src.length - 2 );
     } else {
         src = src.substr( 4 );
-        src = src.substr( 0, src.length - 1 );  
+        src = src.substr( 0, src.length - 1 );
     }
 
-    return src;  
+    return src;
 };
 
 _html2canvas.Util.Bounds = function getBounds (el) {
     var clientRect,
     bounds = {};
-        
-    if (el.getBoundingClientRect){	
+
+    if (el.getBoundingClientRect){
         clientRect = el.getBoundingClientRect();
 
-            
+
         // TODO add scroll position to bounds, so no scrolling of window necessary
         bounds.top = clientRect.top;
         bounds.bottom = clientRect.bottom || (clientRect.top + clientRect.height);
         bounds.left = clientRect.left;
-        
+
         // older IE doesn't have width/height, but top/bottom instead
         bounds.width = clientRect.width || (clientRect.right - clientRect.left);
         bounds.height = clientRect.height || (clientRect.bottom - clientRect.top);
-    
+
         return bounds;
-            
-    }       
+
+    }
 };
 
 _html2canvas.Util.getCSS = function (el, attribute) {
     // return $(el).css(attribute);
-    
+
     var val;
-    
+
     function toPX( attribute, val ) {
         var rsLeft = el.runtimeStyle && el.runtimeStyle[ attribute ],
         left,
         style = el.style;
-                
+
         // Check if we are not dealing with pixels, (Opera has issues with this)
         // Ported from jQuery css.js
         // From the awesome hack by Dean Edwards
         // http://erik.eae.net/archives/2007/07/27/18.54.15/#comment-102291
 
         // If we're not dealing with a regular pixel number
-        // but a number that has a weird ending, we need to convert it to pixels   
-        
+        // but a number that has a weird ending, we need to convert it to pixels
+
         if ( !/^-?[0-9]+\.?[0-9]*(?:px)?$/i.test( val ) && /^-?\d/.test( val ) ) {
 
             // Remember the original values
@@ -95,37 +95,37 @@ _html2canvas.Util.getCSS = function (el, attribute) {
             }
 
         }
-        
+
         if (!/^(thin|medium|thick)$/i.test( val )) {
             return Math.round(parseFloat( val )) + "px";
-        } 
-        
+        }
+
         return val;
-     
+
     }
-    
-    
+
+
     if ( window.getComputedStyle ) {
         val = document.defaultView.getComputedStyle(el, null)[ attribute ];
-        
+
         if ( attribute === "backgroundPosition" ) {
-            
+
             val = (val.split(",")[0] || "0 0").split(" ");
-          
+
             val[ 0 ] = ( val[0].indexOf( "%" ) === -1 ) ? toPX(  attribute + "X", val[ 0 ] ) : val[ 0 ];
             val[ 1 ] = ( val[1] === undefined ) ? val[0] : val[1]; // IE 9 doesn't return double digit always
             val[ 1 ] = ( val[1].indexOf( "%" ) === -1 ) ? toPX(  attribute + "Y", val[ 1 ] ) : val[ 1 ];
-        } 
-        
+        }
+
     } else if ( el.currentStyle ) {
-        // IE 9>       
+        // IE 9>
         if (attribute === "backgroundPosition") {
-            // Older IE uses -x and -y 
-            val = [ toPX(  attribute + "X", el.currentStyle[ attribute + "X" ]  ), toPX(  attribute + "Y", el.currentStyle[ attribute + "Y" ]  ) ];  
+            // Older IE uses -x and -y
+            val = [ toPX(  attribute + "X", el.currentStyle[ attribute + "X" ]  ), toPX(  attribute + "Y", el.currentStyle[ attribute + "Y" ]  ) ];
         } else {
 
             val = toPX(  attribute, el.currentStyle[ attribute ]  );
-            
+
             if (/^(border)/i.test( attribute ) && /^(medium|thin|thick)$/i.test( val )) {
                 switch (val) {
                     case "thin":
@@ -138,7 +138,7 @@ _html2canvas.Util.getCSS = function (el, attribute) {
                         val = "5px";
                         break;
                 }
-            }    
+            }
         }
 
 
@@ -147,61 +147,61 @@ _html2canvas.Util.getCSS = function (el, attribute) {
 
 
 
-    
+
     return val;
-    
 
-    
+
+
 //return $(el).css(attribute);
-    
-  
+
+
 };
 
 
 _html2canvas.Util.BackgroundPosition = function ( el, bounds, image ) {
     // TODO add support for multi image backgrounds
-    
+
     var bgposition =  _html2canvas.Util.getCSS( el, "backgroundPosition" ) ,
     topPos,
     left,
     percentage,
     val;
-    
+
     if (bgposition.length === 1){
         val = bgposition;
-            
+
         bgposition = [];
-        
+
         bgposition[0] = val;
         bgposition[1] = val;
-    }  
+    }
 
-    
 
-    if (bgposition[0].toString().indexOf("%") !== -1){    
-        percentage = (parseFloat(bgposition[0])/100);        
+
+    if (bgposition[0].toString().indexOf("%") !== -1){
+        percentage = (parseFloat(bgposition[0])/100);
         left =  ((bounds.width * percentage)-(image.width*percentage));
-      
+
     }else{
         left = parseInt(bgposition[0],10);
     }
 
-    if (bgposition[1].toString().indexOf("%") !== -1){  
+    if (bgposition[1].toString().indexOf("%") !== -1){
 
-        percentage = (parseFloat(bgposition[1])/100);     
+        percentage = (parseFloat(bgposition[1])/100);
         topPos =  ((bounds.height * percentage)-(image.height*percentage));
-    }else{      
-        topPos = parseInt(bgposition[1],10);      
+    }else{
+        topPos = parseInt(bgposition[1],10);
     }
 
-    
 
-           
+
+
     return {
         top: topPos,
         left: left
     };
-         
+
 };
 
 _html2canvas.Util.Extend = function (options, defaults) {
@@ -219,7 +219,7 @@ _html2canvas.Util.Children = function(el) {
     try {
         children = $(el).contents();
     //children = (el.nodeName && el.nodeName.toUpperCase() === "IFRAME") ? el.contentDocument || el.contentWindow.document :  el.childNodes ;
-       
+
     } catch (ex) {
         h2clog("html2canvas.Util.Children failed with exception: " + ex.message);
         children = [];