We're a real project now

.jscsrc

@@ -11,6 +11,6 @@
     "requireSpacesInConditionalExpression": true,
     "requireSpaceAfterLineComment": true,
     "requirePaddingNewLinesInObjects": true,
-    "requireNewlineBeforeBlockStatements": true,
+    "requireNewlineBeforeBlockStatements": false,
     "requireBlocksOnNewline": true
 }

.jshintrc

@@ -13,8 +13,7 @@
         "require"   : false,
         "module"    : false,
         "console"   : false,
-        "File"      : false,
-        "FrameByFrame": false
+        "File"      : false
     }
 }
 

CHANGELOG.md

@@ -1,3 +1,8 @@
 # 2016-02-24
 
+- made this into an actual project now, bower, npm support
+- we'll call this 1.0
+
+# 2016-02-24
+
 - @miguelgrinberg added support for VideoJS 5.X

Gruntfile.js

@@ -0,0 +1,68 @@
+/*global module:false*/
+module.exports = function(grunt) {
+
+  // Project configuration.
+  grunt.initConfig({
+    // Metadata.
+    pkg: grunt.file.readJSON('package.json'),
+    banner: '/*! <%= pkg.title || pkg.name %> v<%= pkg.version %>\n' +
+      '<%= pkg.homepage ? "* " + pkg.homepage + "\\n" : "" %>' +
+      '* Copyright (c) Eric Rasche 2015-<%= grunt.template.today("yyyy") %>' +
+      ' - Licensed <%= pkg.license %> */\n',
+    // Task configuration.
+    concat: {
+      options: {
+        banner: '<%= banner %>',
+        stripBanners: true
+      },
+      dist: {
+        src: ['<%= pkg.main %>'],
+        dest: 'dist/<%= pkg.main %>'
+      }
+    },
+    uglify: {
+      options: {
+        banner: '<%= banner %>'
+      },
+      dist: {
+        src: '<%= concat.dist.dest %>',
+        dest: 'dist/videojs.framebyframe.min.js'
+      }
+    },
+    jshint: {
+      src: {
+        options: {
+          jshintrc: '.jshintrc'
+        },
+        src: ['<%= pkg.main %>']
+      },
+    },
+    jscs: {
+      src: ['<%= concat.dist.src %>'],
+      options: {
+        config: '.jscsrc',
+        esnext: false, // If you use ES6 http://jscs.info/overview.html#esnext
+        verbose: true, // If you need output with rule names http://jscs.info/overview.html#verbose
+        fix: false, // Autofix code style violations when possible.
+        requireCurlyBraces: [ "if" ]
+      }
+    },
+    watch: {
+      gruntfile: {
+        files: '<%= jshint.gruntfile.src %>',
+        tasks: ['jshint:gruntfile']
+      }
+    }
+  });
+
+  // These plugins provide necessary tasks.
+  grunt.loadNpmTasks('grunt-contrib-concat');
+  grunt.loadNpmTasks('grunt-contrib-uglify');
+  grunt.loadNpmTasks('grunt-contrib-jshint');
+  grunt.loadNpmTasks('grunt-contrib-watch');
+  grunt.loadNpmTasks('grunt-jscs');
+
+  // Default task.
+  grunt.registerTask('default', ['jshint', 'jscs', 'concat', 'uglify']);
+
+};

bower.json

@@ -0,0 +1,24 @@
+{
+  "name": "videojs-framebyframe",
+  "version": "1.0.0",
+  "homepage": "https://github.com/erasche/videojs-framebyframe",
+  "description": "video.js plugin that allows users to step frame-by-frame through a video.",
+  "main": "videojs.framebyframe.js",
+  "keywords": [
+    "video",
+    "videojs",
+    "videojs-plugin"
+  ],
+  "license": "GPL3",
+  "ignore": [
+    ".gitignore",
+    "node_modules",
+    "bower_components",
+    "README.md",
+    "CHANGELOG.md",
+    "examples"
+  ],
+  "dependencies": {
+    "video.js": ">=5.3.0",
+  }
+}

package.json

@@ -0,0 +1,36 @@
+{
+  "name": "videojs-framebyframe",
+  "version": "1.0.0",
+  "description": "video.js plugin that allows users to step frame-by-frame through a video.",
+  "main": "videojs.framebyframe.js",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/erasche/videojs-framebyframe.git"
+  },
+  "scripts": {
+    "test": "echo No tests yet!"
+  },
+  "keywords": [
+    "video",
+    "videojs",
+    "videojs-plugin"
+  ],
+  "dependencies": {
+    "video.js": ">=5.3.0"
+  },
+  "devDependencies": {
+    "grunt": ">=0.4.5",
+    "grunt-contrib-concat": ">=0.4.0",
+    "grunt-contrib-jshint": ">=0.10.0",
+    "grunt-contrib-uglify": ">=0.5.0",
+    "grunt-contrib-watch": ">=0.6.1",
+    "grunt-jscs": "^2.7.0"
+  },
+  "author": "Eric Rasche",
+  "license": "GPL3",
+  "readmeFilename": "README.md",
+  "bugs": {
+    "url": "https://github.com/erasche/videojs-framebyframe/issues"
+  },
+  "homepage": "https://github.com/erasche/videojs-framebyframe"
+}

video-framebyframe.js → videojs.framebyframe.js

@@ -11,7 +11,7 @@ var FBFButton = videojs.extend(VjsButton, {
     },
 
     onClick: function() {
-        //Start by pausing the player
+        // Start by pausing the player
         this.player.pause();
         // Calculate movement distance
         var dist = this.frameTime * this.step_size;
@@ -43,5 +43,5 @@ function framebyframe(options) {
             {}, opt.index);
         });
     });
-};
+}
 videojs.plugin('framebyframe', framebyframe);