From 23e966ac2ef3383de4ef3cdd0fda04a1d31704dd Mon Sep 17 00:00:00 2001
From: Dominic Barnes <dbarnes@treetopllc.com>
Date: Tue, 21 Jan 2014 10:32:34 -0600
Subject: [PATCH 1/3] using component/bind for Function#bind

---
 component.json  |  1 +
 index.js        | 13 +++++++------
 test/index.html |  3 ++-
 3 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/component.json b/component.json
index 3d621d4..a180b56 100644
--- a/component.json
+++ b/component.json
@@ -7,6 +7,7 @@
     "ui"
   ],
   "dependencies": {
+    "component/bind": "*",
     "component/emitter": "*",
     "component/dom": "*"
   },
diff --git a/index.js b/index.js
index 07d9b50..5f6e0a4 100644
--- a/index.js
+++ b/index.js
@@ -3,7 +3,8 @@
  * Module dependencies.
  */
 
-var Emitter = require('emitter')
+var bind    = require('bind')
+  , Emitter = require('emitter')
   , dom     = require('dom')
 
 /**
@@ -33,10 +34,10 @@ function Menu() {
   this.menu = dom('<ul class=menu>').css('display','none');
   this.el = this.menu[0];
   document.body.appendChild(this.el);
-  this.menu.on('hover', this.deselect.bind(this));
-  document.getElementsByTagName('html')[0].onclick = this.hide.bind(this);
-  this.on('show', this.bindKeyboardEvents.bind(this));
-  this.on('hide', this.unbindKeyboardEvents.bind(this));
+  this.menu.on('hover', bind(this, this.deselect));
+  document.getElementsByTagName('html')[0].onclick = bind(this, this.hide);
+  this.on('show', bind(this, this.bindKeyboardEvents));
+  this.on('hide', bind(this, this.unbindKeyboardEvents));
 }
 
 /**
@@ -62,7 +63,7 @@ Menu.prototype.deselect = function(){
  */
 
 Menu.prototype.bindKeyboardEvents = function(){
-  dom(document).on('keydown', this._fnKeyDown = this.onkeydown.bind(this));
+  dom(document).on('keydown', this._fnKeyDown = bind(this, this.onkeydown));
   return this;
 };
 
diff --git a/test/index.html b/test/index.html
index 6f4e0fb..f57bc58 100644
--- a/test/index.html
+++ b/test/index.html
@@ -10,6 +10,7 @@
     <script src="../build/build.js"></script>
     <script>
       var menu = require('menu');
+      var bind = require('bind');
 
       menu = menu();
 
@@ -17,7 +18,7 @@
       .add('add-item', 'Add <em>item</em>')
       .add('Edit item', function(){ console.log('edit'); })
       .add('Remove item', function(){ console.log('remove'); })
-      .add('Clear Menu', menu.clear.bind(menu))
+      .add('Clear Menu', bind(menu, menu.clear))
       .add('Remove "Add item"', function(){
         menu.remove('add-item');
         menu.remove('Remove "Add item"');

From cb7875685d8364a440552c673ff6c8a661755b71 Mon Sep 17 00:00:00 2001
From: Dominic Barnes <dbarnes@treetopllc.com>
Date: Tue, 28 Jan 2014 14:07:56 -0600
Subject: [PATCH 2/3] improving makefile

---
 Makefile | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Makefile b/Makefile
index 00bb7f9..4d6f783 100644
--- a/Makefile
+++ b/Makefile
@@ -1,9 +1,9 @@
 
 build: index.js components
-	@component build
+	@component build --dev
 
-components:
-	@component install
+components: component.json
+	@component install --dev
 
 clean:
 	rm -fr build components

From 192b5942d14710328fe617396a9e5af017845788 Mon Sep 17 00:00:00 2001
From: Dominic Barnes <dbarnes@treetopllc.com>
Date: Tue, 28 Jan 2014 14:08:15 -0600
Subject: [PATCH 3/3] adding event normalization and fixes for test/example
 page

---
 component.json  |  6 +++++-
 index.js        | 14 ++++++++------
 test/index.html | 10 ++++++----
 3 files changed, 19 insertions(+), 11 deletions(-)

diff --git a/component.json b/component.json
index a180b56..659d64a 100644
--- a/component.json
+++ b/component.json
@@ -8,8 +8,12 @@
   ],
   "dependencies": {
     "component/bind": "*",
+    "component/dom": "*",
     "component/emitter": "*",
-    "component/dom": "*"
+    "stephenmathieson/normalize": "*"
+  },
+  "development": {
+    "yields/prevent": "*"
   },
   "styles": [
     "menu.css"
diff --git a/index.js b/index.js
index 5f6e0a4..713402d 100644
--- a/index.js
+++ b/index.js
@@ -3,9 +3,10 @@
  * Module dependencies.
  */
 
-var bind    = require('bind')
-  , Emitter = require('emitter')
-  , dom     = require('dom')
+var bind      = require('bind')
+  , Emitter   = require('emitter')
+  , dom       = require('dom')
+  , normalize = require('normalize')
 
 /**
  * Expose `Menu`.
@@ -158,14 +159,14 @@ Menu.prototype.add = function(text, fn){
              .addClass('menu-item-' + slug)
 
   el.find('a')
-    .on('click', function(e){
+    .on('click', normalize(function(e){
       e.preventDefault();
       e.stopPropagation();
       self.hide();
       self.emit('select', slug);
       self.emit(slug);
       fn && fn();
-    });
+    }));
 
   this.el.appendChild(el[0]);
   this.items[slug] = el[0];
@@ -226,7 +227,8 @@ Menu.prototype.has = function(slug){
  */
 
 Menu.prototype.moveTo = function(x, y){
-  this.menu.css('top', y).css('left',x);
+  this.menu.css('top', y);
+  this.menu.css('left', x);
   return this;
 };
 
diff --git a/test/index.html b/test/index.html
index f57bc58..705f68b 100644
--- a/test/index.html
+++ b/test/index.html
@@ -11,6 +11,8 @@
     <script>
       var menu = require('menu');
       var bind = require('bind');
+      var normalize = require('normalize');
+      var prevent = require('prevent');
 
       menu = menu();
 
@@ -37,11 +39,11 @@
         menu.add('born-again', '<em>Born Again</em>');
       });
 
-      oncontextmenu = function(e){
-        e.preventDefault();
-        menu.moveTo(e.pageX, e.pageY);
+      document.oncontextmenu = normalize(function(e) {
+        prevent(e);
+        menu.moveTo(e.clientX, e.clientY);
         menu.show();
-      };
+      });
     </script>
   </body>
 </html>