Add editorconfig and cleanup the indentation

kdzwinel · kdzwinel · commit 64cfe41f6d27 · 2015-08-07T01:09:25.000+02:00
diff --git a/.editorconfig b/.editorconfig
@@ -0,0 +1,17 @@
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+charset = utf-8
+
+[*.json]
+indent_style = space
+indent_size = 2
+
+[*.html]
+indent_style = space
+indent_size = 4
+
+[*.{js,css}]
+indent_style = tab
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -1,4 +1,4 @@
-module.exports = function(grunt) {
+module.exports = function (grunt) {
 	"use strict";
 
 	grunt.initConfig({
@@ -49,4 +49,4 @@ module.exports = function(grunt) {
 
 	grunt.registerTask('default', ['jshint', 'csslint', 'validation']);
 	grunt.registerTask('prod', ['zip']);
-};
+};
diff --git a/css/panel.css b/css/panel.css
@@ -41,7 +41,7 @@ iframe {
 	right: 0;
 	top: 0;
 	bottom: 0;
-	background: rgba(0,0,0,0.5);
+	background: rgba(0, 0, 0, 0.5);
 	opacity: 0;
 	z-index: 2;
 	-webkit-transition: opacity 0.3s, visibility 0.3s;
@@ -98,14 +98,13 @@ iframe {
 	z-index: 1;
 	display: block;
 
-	background: rgba(255,255,255,0.75);
-	-webkit-border-top-left-radius: 5px;
+	background: rgba(255, 255, 255, 0.75);
 	border-top-left-radius: 5px;
 	padding-left: 5px;
 	padding-top: 5px;
 }
 
-	#social-buttons iframe {
-		margin-top: 3px;
-		display: block;
-	}
+#social-buttons iframe {
+	margin-top: 3px;
+	display: block;
+}
diff --git a/devtools.html b/devtools.html
@@ -6,4 +6,4 @@
 </head>
 <body>
 </body>
-</html>
+</html>
diff --git a/js/background.js b/js/background.js
@@ -1,17 +1,17 @@
 /*
-Allows to read, change and override settings kept in localStorage
+ Allows to read, change and override settings kept in localStorage
 
-FIXME Can be replaced with chrome.storage.local as soon as http://crbug.com/178618 will be resolved
-FIXME Can be replaced with localStorage on the panel page as soon as http://crbug.com/319328 will be resolved
+ FIXME Can be replaced with chrome.storage.local as soon as http://crbug.com/178618 will be resolved
+ FIXME Can be replaced with localStorage on the panel page as soon as http://crbug.com/319328 will be resolved
  */
-chrome.runtime.onMessage.addListener(function(message, sender, callback){
+chrome.runtime.onMessage.addListener(function (message, sender, callback) {
 	"use strict";
 
-	if(message.name === 'getSettings') {
+	if (message.name === 'getSettings') {
 		callback(localStorage);
-	} else if(message.name === 'setSettings') {
+	} else if (message.name === 'setSettings') {
 		localStorage = message.data;
-	} else if(message.name === 'changeSetting') {
+	} else if (message.name === 'changeSetting') {
 		localStorage[message.item] = message.value;
 	}
-});
+});
diff --git a/js/devtools.js b/js/devtools.js
@@ -1,2 +1,2 @@
 chrome.devtools.panels.create("SnappySnippet", "gfx/devtools-logo.png", "panel.html", function (panel) {
-});
+});
diff --git a/js/filters/DefaultValueFilter.js b/js/filters/DefaultValueFilter.js
@@ -70,4 +70,4 @@ function DefaultValueFilter() {
 
 		return output;
 	};
-}
+}
diff --git a/js/filters/ShorthandPropertyFilter.js b/js/filters/ShorthandPropertyFilter.js
@@ -84,7 +84,7 @@ function ShorthandPropertyFilter() {
 				for (i = 0, l = longhands.length; i < l; i++) {
 					blacklist[longhands[i]] = true;
 				}
-			} else if(!style[shorthand]) {
+			} else if (!style[shorthand]) {
 				blacklist[shorthand] = true;
 			}
 		}
@@ -117,4 +117,4 @@ function ShorthandPropertyFilter() {
 
 		return output;
 	};
-}
+}
diff --git a/js/filters/WebkitPropertiesFilter.js b/js/filters/WebkitPropertiesFilter.js
@@ -38,4 +38,4 @@ function WebkitPropertiesFilter() {
 
 		return output;
 	};
-}
+}
diff --git a/js/panel.js b/js/panel.js
@@ -33,7 +33,7 @@
 	restoreSettings();
 
 	/*
-	SUBMITTING THE CODE TO CodePen/jsFiddle/jsBin
+	 SUBMITTING THE CODE TO CodePen/jsFiddle/jsBin
 	 */
 
 	codepenForm.on('submit', function () {
@@ -62,7 +62,7 @@
 	});
 
 	/*
-	Event listeners
+	 Event listeners
 	 */
 
 	propertiesCleanUpInput.on('change', persistSettingAndProcessSnapshot);
@@ -91,15 +91,15 @@
 		return validator.test(prefix);
 	}
 
-	idPrefix.on('change', function() {
+	idPrefix.on('change', function () {
 		var val = $(this).val(),
 			parent = $(this).parent();
 
 		parent.removeClass('has-error').removeClass('has-success');
 
-		if(val.length === 0) {
+		if (val.length === 0) {
 			persistSettingAndProcessSnapshot.apply(this);
-		} else if(isValidPrefix(val)) {
+		} else if (isValidPrefix(val)) {
 			parent.addClass('has-success');
 			persistSettingAndProcessSnapshot.apply(this);
 		} else {
@@ -108,7 +108,7 @@
 	});
 
 	/*
-	Settings - saving & restoring
+	 Settings - saving & restoring
 	 */
 
 	function restoreSettings() {
@@ -126,7 +126,7 @@
 					continue;
 				}
 
-				if(el.is('[type=checkbox]')) {
+				if (el.is('[type=checkbox]')) {
 					//updating flat UI checkbox
 					el.data('checkbox').setCheck(settings[prop] === "true" ? 'check' : 'uncheck');
 				} else {
@@ -155,7 +155,7 @@
 	}
 
 	/*
-	Making & processing snippets
+	 Making & processing snippets
 	 */
 
 	function makeSnapshot() {
@@ -228,7 +228,7 @@
 
 		styles = cssStringifier.process(styles);
 
-		if(isValidPrefix(idPrefix.val())) {
+		if (isValidPrefix(idPrefix.val())) {
 			prefix = idPrefix.val();
 		}
 
diff --git a/js/tools/CSSStringifier.js b/js/tools/CSSStringifier.js
@@ -67,4 +67,4 @@ function CSSStringifier() {
 
 		return output;
 	};
-}
+}
diff --git a/js/tools/InspectedContext.js b/js/tools/InspectedContext.js
@@ -64,4 +64,4 @@ function InspectedContext() {
 			chrome.devtools.inspectedWindow.eval(code, settings, callback);
 		});
 	};
-}
+}
diff --git a/js/tools/SameRulesCombiner.js b/js/tools/SameRulesCombiner.js
@@ -50,4 +50,4 @@ function SameRulesCombiner() {
 
 		return output;
 	};
-}
+}
diff --git a/js/tools/Snapshooter.js b/js/tools/Snapshooter.js
@@ -87,15 +87,15 @@ function Snapshooter(root) {
 
 		output = [];
 
-		if(content) {
+		if (content) {
 			//content property can take multiple values - we need to split them up
 			//FIXME this won't work for '\''
 			values = content.match(/(?:[^\s']+|'[^']*')+/g);
 
-			for(i=0, l=values.length; i<l; i++) {
+			for (i = 0, l = values.length; i < l; i++) {
 				value = values[i];
 
-				if(value.match(/^(url\()|(attr\()|normal|none|open-quote|close-quote|no-open-quote|no-close-quote|chapter_counter|'/g)) {
+				if (value.match(/^(url\()|(attr\()|normal|none|open-quote|close-quote|no-open-quote|no-close-quote|chapter_counter|'/g)) {
 					output.push(value);
 				} else {
 					output.push("'" + value + "'");
@@ -170,36 +170,36 @@ function Snapshooter(root) {
 		return result;
 	}
 
-    /**
-     * Replaces all relative URLs (in images, links etc.) with absolute URLs
-     * @param element
-     */
-    function relativeURLsToAbsoluteURLs(element) {
-        switch(element.nodeName) {
-            case 'A':
-            case 'AREA':
-            case 'LINK':
-            case 'BASE':
-                if(element.hasAttribute('href')) {
-                    element.setAttribute('href', element.href);
-                }
-                break;
-            case 'IMG':
-            case 'IFRAME':
-            case 'INPUT':
-            case 'FRAME':
-            case 'SCRIPT':
-                if(element.hasAttribute('src')) {
-                    element.setAttribute('src', element.src);
-                }
-                break;
-            case 'FORM':
-                if(element.hasAttribute('action')) {
-                    element.setAttribute('action', element.action);
-                }
-                break;
-        }
-    }
+	/**
+	 * Replaces all relative URLs (in images, links etc.) with absolute URLs
+	 * @param element
+	 */
+	function relativeURLsToAbsoluteURLs(element) {
+		switch (element.nodeName) {
+			case 'A':
+			case 'AREA':
+			case 'LINK':
+			case 'BASE':
+				if (element.hasAttribute('href')) {
+					element.setAttribute('href', element.href);
+				}
+				break;
+			case 'IMG':
+			case 'IFRAME':
+			case 'INPUT':
+			case 'FRAME':
+			case 'SCRIPT':
+				if (element.hasAttribute('src')) {
+					element.setAttribute('src', element.src);
+				}
+				break;
+			case 'FORM':
+				if (element.hasAttribute('action')) {
+					element.setAttribute('action', element.action);
+				}
+				break;
+		}
+	}
 
 	function init() {
 		var css = [],
@@ -244,7 +244,7 @@ function Snapshooter(root) {
 		for (i = 0, l = descendants.length; i < l; i++) {
 			descendant = descendants[i];
 			descendant.setAttribute('id', createID(descendant));
-            relativeURLsToAbsoluteURLs(descendant);
+			relativeURLsToAbsoluteURLs(descendant);
 		}
 
 		// Build leading and trailing HTML for ancestors
diff --git a/manifest.json b/manifest.json
@@ -1,17 +1,19 @@
 {
-	"name": "SnappySnippet",
-	"description": "Easily extract CSS and HTML from selected element. Then send it to CodePen, jsFiddle or JS Bin with one click.",
-	"version": "0.5",
-	"manifest_version": 2,
-	"minimum_chrome_version": "18.0",
-	"icons": {
-		"24": "gfx/icon_24.png",
-		"128": "gfx/icon_128.png"
-	},
-	"devtools_page": "devtools.html",
-	"background": {
-		"scripts": ["js/background.js"],
-		"persistent": false
-	},
-	"content_security_policy": "script-src 'self' https://apis.google.com https://platform.twitter.com https://www.facebook.com; object-src 'self'"
-}
+  "name": "SnappySnippet",
+  "description": "Easily extract CSS and HTML from selected element. Then send it to CodePen, jsFiddle or JS Bin with one click.",
+  "version": "0.5",
+  "manifest_version": 2,
+  "minimum_chrome_version": "18.0",
+  "icons": {
+    "24": "gfx/icon_24.png",
+    "128": "gfx/icon_128.png"
+  },
+  "devtools_page": "devtools.html",
+  "background": {
+    "scripts": [
+      "js/background.js"
+    ],
+    "persistent": false
+  },
+  "content_security_policy": "script-src 'self' https://apis.google.com https://platform.twitter.com https://www.facebook.com; object-src 'self'"
+}

-Original file line number
+Diff line change
 </head>
 <body>
 </body>
 -</html>
 +</html>
Original file line number	Diff line number	Diff line change
`@@ -1,2 +1,2 @@`
`1`	`1`	`chrome.devtools.panels.create("SnappySnippet", "gfx/devtools-logo.png", "panel.html", function (panel) {`
`2`		`-});`
	`2`	`+});`
Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,7 @@ function ShorthandPropertyFilter() {`
`84`	`84`	`for (i = 0, l = longhands.length; i < l; i++) {`
`85`	`85`	`blacklist[longhands[i]] = true;`
`86`	`86`	`}`
`87`		`- } else if(!style[shorthand]) {`
	`87`	`+ } else if (!style[shorthand]) {`
`88`	`88`	`blacklist[shorthand] = true;`
`89`	`89`	`}`
`90`	`90`	`}`
`@@ -117,4 +117,4 @@ function ShorthandPropertyFilter() {`
`117`	`117`
`118`	`118`	`return output;`
`119`	`119`	`};`
`120`		`-}`
	`120`	`+}`