Sarting to drop Opera 12 support.

Voog · Apr 25, 2014 · bae6f56 · bae6f56
1 parent 9b68b40
commit bae6f56
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 23 deletions.
diff --git a/src/browser.js b/src/browser.js
@@ -278,11 +278,6 @@ wysihtml5.browser = (function() {
       return "getSelection" in window && "modify" in window.getSelection();
     },
 
-    // Returns if there is a way for setting selection to expand a line
-    supportsSelectLine: function () {
-        return (this.supportsSelectionModify() || document.selection) ? true : false;
-    },
-
     /**
      * Opera needs a white space after a <br> in order to position the caret correctly
      */

diff --git a/src/commands/formatBlock.js b/src/commands/formatBlock.js
@@ -262,24 +262,11 @@
         }
       }
 
-      if (wysihtml5.browser.supportsSelectLine()) {
-          _selectionWrap(composer, {
-            "nodeName": (nodeName || defaultNodeName),
-            "className": className || null,
-            "cssStyle": cssStyle || null
-          });
-      } else {
-          // Falling back to native command for Opera up to 12 mostly
-          // Native command does not create elements from selecton boundaries.
-          // Not quite user expected behaviour
-          // TODO: Also it does not support adding style. Drop opera 12 support or find a fallback
-          if (composer.commands.support(command)) {
-            _execCommand(doc, composer, command, nodeName || defaultNodeName, className);
-            return;
-          }
-      }
-
-
+      _selectionWrap(composer, {
+        "nodeName": (nodeName || defaultNodeName),
+        "className": className || null,
+        "cssStyle": cssStyle || null
+      });
     },
 
     state: function(composer, command, nodeName, className, classRegExp, cssStyle, styleRegExp) {