Improved build

Author: mar10

Gruntfile.js

@@ -6,17 +6,17 @@ module.exports = function (grunt) {
 
 	grunt.initConfig({
 		pkg: grunt.file.readJSON("package.json"),
-        exec: {
-            tabfix: {
-                // Cleanup whitespace according to http://contribute.jquery.org/style-guide/js/
-                // (requires https://github.com/mar10/tabfix)
-                cmd: "tabfix -t -r -m*.js,*.css,*.html src demo test"
-            },
-            upload: {
-                // FTP upload the demo files (requires https://github.com/mar10/pyftpsync)
-                cmd: "pyftpsync upload . ftp://www.wwwendt.de/tech/demo/jquery-contextmenu --delete-unmatched --omit dist,node_modules,.*,_* -x"
-            }
-        },
+		exec: {
+			tabfix: {
+				// Cleanup whitespace according to http://contribute.jquery.org/style-guide/js/
+				// (requires https://github.com/mar10/tabfix)
+				cmd: "tabfix -t -r -m *.js,*.css,*.html,*json,*.yaml -i node_modules ."
+			},
+			upload: {
+				// FTP upload the demo files (requires https://github.com/mar10/pyftpsync)
+				cmd: "pyftpsync --progress upload . ftp://www.wwwendt.de/tech/demo/jquery-contextmenu --delete-unmatched --omit dist,node_modules,.*,_* -x"
+			}
+		},
 		qunit: {
 			all: ["test/index.html"]
 		},
@@ -32,28 +32,33 @@ module.exports = function (grunt) {
 		uglify: {
 			options: {
 //				banner: "/*! <%= pkg.name %> v<%= pkg.version %> | <%= pkg.license %> */\n"
-	            banner: "/*! <%= pkg.title || pkg.name %> - v<%= pkg.version %> - " +
-                "<%= grunt.template.today('yyyy-mm-dd') %> | " +
-                "<%= pkg.homepage ? ' ' + pkg.homepage + ' | ' : '' %>" +
-                " Copyright (c) <%= grunt.template.today('yyyy') %> <%= pkg.author.name %>;" +
-                " Licensed <%= _.pluck(pkg.licenses, 'type').join(', ') %> */\n"
+				banner: "/*! <%= pkg.title || pkg.name %> - v<%= pkg.version %> - " +
+				"<%= grunt.template.today('yyyy-mm-dd') %> | " +
+				"<%= pkg.homepage ? ' ' + pkg.homepage + ' | ' : '' %>" +
+				" Copyright (c) <%= grunt.template.today('yyyy') %> <%= pkg.author.name %>;" +
+				" Licensed <%= _.pluck(pkg.licenses, 'type').join(', ') %> */\n"
 			},
 			build: {
 				src: "jquery.contextmenu.js",
 //                dest: "build/jquery.contextmenu-<%= pkg.version %>.min.js"
-                dest: "jquery.contextmenu.min.js"
+				dest: "jquery.contextmenu.min.js"
 			}
 		}
 	});
 
 	grunt.loadNpmTasks("grunt-contrib-jshint");
 	grunt.loadNpmTasks("grunt-contrib-uglify");
 	grunt.loadNpmTasks("grunt-contrib-qunit");
-    grunt.loadNpmTasks("grunt-exec");
+	grunt.loadNpmTasks("grunt-exec");
 
-    grunt.registerTask("ci", ["jshint", "qunit"]);
-	grunt.registerTask("default", ["jshint", "qunit", "uglify"]);
-    grunt.registerTask("upload", [//"build",
-                                  "exec:upload"]);
-    grunt.registerTask("build", ["exec:tabfix", "default"]);
+	// The 'ci' task is run on travis
+	grunt.registerTask("ci", ["jshint",
+							  "qunit"]);
+	grunt.registerTask("default", ["jshint",
+								   "qunit",
+								   "uglify"]);
+	grunt.registerTask("build", ["exec:tabfix",
+								 "default"]);
+	grunt.registerTask("upload", ["build",
+								  "exec:upload"]);
 };

README.md

@@ -2,19 +2,20 @@
 
 A jQuery plugin that provides a context menu (based on the standard [jQueryUI menu] widget).
 
-  * themable
-  * supports delegation (i.e. can be bound to elements that don't exist at the
-    time the context menu is initialized)
-  * exposes events from [jQueryUI menu]: `blur`, `create`, `focus`, `select`
+  * Themable using [jQuery ThemeRoller](http://jqueryui.com/themeroller/).
+  * Supports delegation (i.e. can be bound to elements that don't exist at the
+    time the context menu is initialized).
+  * Exposes events from [jQueryUI menu]: `blur`, `create`, `focus`, `select`.
 
 
 ## Status
+
 Beta. Please report issues.
 
 
 ## Demo
 
-[Live demo page](http://mar10.github.com/jquery-contextmenu/demo/sample-widget.html)
+[Live demo page](http://wwwendt.de/tech/demo/jquery-contextmenu/demo/)
 
 
 ## Example

demo/index.html

@@ -12,23 +12,23 @@
 <style type="text/css">
 body{
 	font-family: "Trebuchet MS", "Helvetica", "Arial",  "Verdana", "sans-serif";
-    font-size: .8em;
+	font-size: .8em;
 }
 .hasmenu{
-    border: 1px solid #008;
-    margin: 3px;
-    padding: 2px;
-    width: 30px;
+	border: 1px solid #008;
+	margin: 3px;
+	padding: 2px;
+	width: 30px;
 }
 .ui-widget{
-    font-size: .8em;
+	font-size: .8em;
 }
 .ui-menu {
 	width: 120px;
 }
 /* Define a custom icon */
 .ui-icon.custom-icon-firefox {
-    background-image: url(application_firefox.gif);
+	background-image: url(application_firefox.gif);
 	background-position: 0 0;
 }
 </style>
@@ -63,7 +63,7 @@
 		// Trigger popup menu on the first target element
 		$("#container").contextmenu("open", $(".hasmenu:first"));
 		setTimeout(function(){
-	        $("#container").contextmenu("close");
+			$("#container").contextmenu("close");
 		}, 2000);
 	});
 });

jquery.contextmenu.js

@@ -208,18 +208,18 @@
 		_setOption: function(key, value){
 			$.Widget.prototype._setOption.apply(this, arguments);
 		},
-        /**
-         * Open context menu on a specific target (must match options.delegate)
-         */
-        open: function(target){
-            var e = jQuery.Event("contextmenu", {target: target.get(0)});
-            return this.element.trigger(e);
-        },
-        /**
-         * Close context menu.
-         */
-        close: function(){
-            return this._closeMenu.call(this);
-        }
+		/**
+		 * Open context menu on a specific target (must match options.delegate)
+		 */
+		open: function(target){
+			var e = jQuery.Event("contextmenu", {target: target.get(0)});
+			return this.element.trigger(e);
+		},
+		/**
+		 * Close context menu.
+		 */
+		close: function(){
+			return this._closeMenu.call(this);
+		}
 	});
 } (jQuery));

package.json

@@ -5,35 +5,35 @@
   "version": "0.1.0",
   "homepage": "https://github.com/mar10/contextmenu",
   "author": {
-    "name": "Martin Wendt",
-    "url": "http://wwwendt.de/"
+	"name": "Martin Wendt",
+	"url": "http://wwwendt.de/"
   },
   "repository": {
-    "type": "svn",
-    "url": "https://github.com/mar10/contextmenu"
+	"type": "svn",
+	"url": "https://github.com/mar10/contextmenu"
   },
   "bugs": {
-    "url": "https://github.com/mar10/contextmenu/issues"
+	"url": "https://github.com/mar10/contextmenu/issues"
   },
   "licenses": [
-    {
-      "type": "MIT",
-      "url": "https://github.com/mar10/jquery-contextmenu/blob/master/MIT-LICENSE.txt"
-    }
+	{
+	  "type": "MIT",
+	  "url": "https://github.com/mar10/jquery-contextmenu/blob/master/MIT-LICENSE.txt"
+	}
   ],
   "dependencies": {
-    "jquery": "~1.8"
+	"jquery": "~1.8"
   },
   "keywords": [],
   "devDependencies": {
-    "grunt": "~0.4.1",
-    "grunt-contrib-jshint": "~0.3.0",
-    "grunt-contrib-uglify": "~0.2.0",
-    "grunt-contrib-qunit": "~0.2.0",
-    "grunt-contrib-concat": "~0.1.3",
-    "grunt-exec": "~0.4.0"
+	"grunt": "~0.4.1",
+	"grunt-contrib-jshint": "~0.3.0",
+	"grunt-contrib-uglify": "~0.2.0",
+	"grunt-contrib-qunit": "~0.2.0",
+	"grunt-contrib-concat": "~0.1.3",
+	"grunt-exec": "~0.4.0"
   },
   "scripts": {
-    "test": "grunt ci --verbose"
+	"test": "grunt ci --verbose"
   }
 }