Added .jscsrc and .jshintrc

flesler · 85708d2a6f020c7e1a9e4b0196897c8e9b09b97d · flesler committed Dec 20, 2014 · Dec 20, 2014 · 85708d2
1 parent 9fafa1c
commit 85708d2a6f020c7e1a9e4b0196897c8e9b09b97d
Unified Split

Showing with 70 additions and 4 deletions.

+41 −0 .jscsrc

+23 −0 .jshintrc

+3 −1 bower.json

+2 −2 jquery.scrollTo.js

+1 −1 package.json
diff --git a/.jscsrc b/.jscsrc
@@ -0,0 +1,41 @@
+{
+	"requireCurlyBraces": [
+		"do","else","for","switch", "try","catch","while"
+	],
+	"requireSpaceBeforeKeywords": [
+		"else", "catch"
+	],
+	"requireSpaceAfterKeywords": [
+		"do", "for", "if", "else", "switch", "case", "try",
+		"catch", "void", "while", "with", "return", "typeof"
+	],
+	"requireSpaceBeforeBlockStatements": true,
+	"requireParenthesesAroundIIFE": true,
+	"requireSpacesInFunctionExpression": {"beforeOpeningCurlyBrace": true},
+	"disallowSpacesInFunctionExpression": {"beforeOpeningRoundBrace": true},
+	"disallowSpacesInCallExpression": true,
+	"requireBlocksOnNewline": 1,
+	"disallowPaddingNewlinesInBlocks": true,
+	"disallowEmptyBlocks": true,
+	"disallowQuotedKeysInObjects": "allButReserved",
+	"disallowSpaceAfterObjectKeys": true,
+	"requireCommaBeforeLineBreak": true,
+	"requireOperatorBeforeLineBreak": true,
+	"disallowSpaceAfterPrefixUnaryOperators": true,
+	"disallowSpaceBeforePostfixUnaryOperators": true,
+	"disallowKeywords": [ "with" ],
+	"disallowMultipleLineBreaks": true,
+	"disallowMixedSpacesAndTabs": "smart",
+	"disallowOperatorBeforeLineBreak": ["."],
+	"disallowTrailingWhitespace": true,
+	"disallowTrailingComma": true,
+	"disallowKeywordsOnNewLine": ["else"],
+	"requireCapitalizedConstructors": true,
+	"requireDotNotation": true,
+	"disallowYodaConditions": true,
+	"disallowNewlineBeforeBlockStatements": true,
+	"validateLineBreaks": "LF",
+	"validateQuoteMarks": { "mark": "'", "escape": true },
+	"validateIndentation": "\t",
+	"validateParameterSeparator": ", "
+}
diff --git a/.jshintrc b/.jshintrc
@@ -0,0 +1,23 @@
+{
+  "shadow": "inner",
+
+  "camelcase": true,
+  "eqeqeq": true,
+  "eqnull": true,
+  "freeze": true,
+  "funcscope": true,
+  "newcap": true,
+  "noarg": true,
+  "noempty": true,
+  "nonbsp": true,
+  "unused": true,
+  "undef": true,
+  "scripturl": true,
+  "strict": true,
+  "quotmark": "single",
+
+  "globals": {"define": true},
+  "browser": true,
+  "node": true,
+  "jquery": true
+}
diff --git a/bower.json b/bower.json
@@ -11,7 +11,9 @@
     "demo",
     "tests",
     "changes.txt",
-    "composer.json"
+    "composer.json",
+    "package.json",
+    "scrollTo.jquery.json"
   ],
   "dependencies": {
     "jquery": ">=1.4"
diff --git a/jquery.scrollTo.js b/jquery.scrollTo.js
@@ -115,8 +115,8 @@
 
 						// If it's a dom element, reduce the margin
 						if (settings.margin) {
-							attr[key] -= parseInt(targ.css('margin'+Pos)) || 0;
-							attr[key] -= parseInt(targ.css('border'+Pos+'Width')) || 0;
+							attr[key] -= parseInt(targ.css('margin'+Pos), 10) || 0;
+							attr[key] -= parseInt(targ.css('border'+Pos+'Width'), 10) || 0;
 						}
 
 						attr[key] += offset[pos] || 0;
diff --git a/package.json b/package.json
@@ -4,7 +4,7 @@
   "description": "Easy element scrolling using jQuery.",
   "main": "jquery.scrollTo.js",
   "license": "MIT",
-  "ignore": ["**/.*","demo","tests","changes.txt","composer.json","scrollTo.jquery.json"],
+  "ignore": ["**/.*","demo","tests","changes.txt","composer.json","bower.json","scrollTo.jquery.json"],
   "dependencies": { "jquery": ">=1.4" },
   "homepage": "https://github.com/flesler/jquery.scrollTo/",
   "bugs": "https://github.com/flesler/jquery.scrollTo/issues",