Fix #2 - no 'use strict' - mixed spaces and tabs - missing radix

parameter and some other jslint errors
micc83 · Dec 3, 2013 · 0c63117 · 0c63117
1 parent d10a6ec
commit 0c63117
Showing 1 changed file with 15 additions and 16 deletions.
diff --git a/jquery.restable.js b/jquery.restable.js
@@ -1,6 +1,8 @@
-/*! ReStable v0.1 by Alessandro Benoit */
+/*! ReStable v0.1.1 by Alessandro Benoit */
 (function ($, window, i) {
 
+    'use strict';
+
     $.fn.ReStable = function (options) {
 
         // Settings
@@ -14,7 +16,7 @@
 
             var $cols = [],
                 $result = {},
-                $cols_header = $(element).find('tr').first().children('td'),
+                $cols_header = $(element).find('tr').first().children('td,th'),
                 $row_number = 0,
                 $list;
 
@@ -31,41 +33,38 @@
             $(element).find('tr').slice(1).each(function () {
 
                 var $row = $(this);
-                $row_number++;
+                $row_number += 1;
 
                 $.each($cols, function (index, value) {
 
-                    index++;
+                    index += 1;
 
                     if (s.rowHeaders) {
-                    	if (!$result[value]) { $result[value] = {}; }
+                        if (!$result[value]) { $result[value] = {}; }
                         $result[value][$row.children('td:nth-child(1)').text()] = $row.children('td:nth-child(' + (index + 1) + ')').text();
                     } else {
-                    	if (!$result[$row_number]) { $result[$row_number] = {}; }
+                        if (!$result[$row_number]) { $result[$row_number] = {}; }
                         $result[$row_number][value] = $row.children('td:nth-child(' + index + ')').text();
                     }
 
                 });
 
             });
-
-            console.log($result);
 
             // Crea la lista
             $list = $('<ul/>', {
                 class: 'tabletolist ' + ((s.rowHeaders) ? 'rh' : 'nrh'),
                 id: 'tabletolist' + i
             }).insertBefore($(element));
             $.each($result, function (index, value) {
-
-                var $myrow = $('<li/>', {
-                    html: (s.rowHeaders)? '<span class="titles">' + index + '</span>' : ''
-                }).appendTo($list);
 
-                var $myrowul = $('<ul/>').appendTo($myrow);
+                var $myrow = $('<li/>', {
+                    html: (s.rowHeaders) ? '<span class="titles">' + index + '</span>' : ''
+                }).appendTo($list),
+                    $myrowul = $('<ul/>').appendTo($myrow);
 
                 $.each(value, function (index, value) {
-					
+
                     $('<li/>', {
                         html: '<span class="row_headers">' + index + '</span> <span class="row_data">' + value + '</span>'
                     }).appendTo($myrowul);
@@ -84,12 +83,12 @@
             var element = $(this),
                 responsive_table;
 
-            i++;
+            i += 1;
 
             // The responsive menu is built if the page size is or goes under maxWidth
             function handle_table() {
 
-                if ($(window).width() > parseInt(s.maxWidth)) {
+                if ($(window).width() > parseInt(s.maxWidth, 10)) {
 
                     $(element).show();