do not move toolbar when page scroll

sofish · Sep 6, 2013 · fda0ea2 · fda0ea2
1 parent 760e1f7
commit fda0ea2
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/src/pen.js b/src/pen.js
@@ -115,7 +115,7 @@
 
   Pen.prototype.toolbar = function() {
 
-    var menu, that = this, icons = '';
+    var menu, that = this, icons = '', setpos;
 
     for(var i = 0, list = this.config.list; i < list.length; i++) {
       var name = list[i], klass = 'pen-icon icon-' + name;
@@ -130,13 +130,13 @@
 
     doc.body.appendChild((this._menu = menu));
 
-    // change menu offset when window resize
-    window.addEventListener('resize', function() {
-      if(menu.style.display === 'block') {
-        menu.style.display = 'none';
-        that.menu();
-      }
-    });
+    setpos = function() {
+      if(menu.style.display === 'block') that.menu();
+    }
+
+    // change menu offset when window resize / scroll
+    window.addEventListener('resize', setpos);
+    window.addEventListener('scroll', setpos);
 
     // show toolbar on select
     this.config.editor.addEventListener('mouseup', function(){