diff --git a/src/mixins/canvas_events.mixin.js b/src/mixins/canvas_events.mixin.js
index dba10979db7..e1bbc4c1a5c 100644
--- a/src/mixins/canvas_events.mixin.js
+++ b/src/mixins/canvas_events.mixin.js
@@ -405,7 +405,7 @@
     __onMouseDown: function (e) {
 
       // accept only left clicks
-      var isLeftClick  = 'which' in e ? e.which === 1 : e.button === 1;
+      var isLeftClick  = 'which' in e ? e.which === 1 : e.button === 0;
       if (!isLeftClick && !fabric.isTouchSupported) {
         return;
       }
diff --git a/src/shapes/itext.class.js b/src/shapes/itext.class.js
index 997d69ea0de..3de05f54d91 100644
--- a/src/shapes/itext.class.js
+++ b/src/shapes/itext.class.js
@@ -699,9 +699,9 @@
 
       var decorationWeight = charHeight / 15,
           positions = {
-            'underline': top + charHeight / 10,
+            underline: top + charHeight / 10,
             'line-through': top - charHeight * (this._fontSizeFraction + this._fontSizeMult - 1) + decorationWeight,
-            'overline': top - (this._fontSizeMult - this._fontSizeFraction) * charHeight
+            overline: top - (this._fontSizeMult - this._fontSizeFraction) * charHeight
           },
           decorations = ['underline', 'line-through', 'overline'], i, decoration;