Cleanup before refactoring to use HubSpot's Tether

README.md

@@ -2,7 +2,7 @@
 
 SelectionMenu is a small, self-contained, framework-agnostic JavaScript module that displays a custom context menu when the user selects text on the page.
 
-This menu may offer a search feature, dictionary lookup, post to Facebook or similar.
+This menu may offer a search feature, dictionary lookup, share on social media, or similar.
 
 **[Live Demo](http://idorecall.github.io/selection-menu/)**
 
@@ -43,7 +43,7 @@ Pass an object literal with the following options:
 * `minlength` (number, optional): Only display the menu if the selected text has at least this length. Defaults to 5 characters.
 * `maxlength` (number, optional): Only display the menu if the selected text is at most this long. Defaults to `Infinity`.
 * `id` (string, optional): The ID of the menu element which is inserted. Defaults to `selection-menu`.
-* `onselect` (function, optional): Custom event generated when the mouse selection changes. Not generated when the selection is changed via the keyboard. Use it to customize the menu dynamically based on the contents of the selection:
+* `onselect` (function, optional): Custom event generated when the mouse/finger selection changes. Not generated when the selection is changed via the keyboard. Use it to customize the menu dynamically based on the contents of the selection:
 
     ```js
     new SelectionMenu({

demos/iDoRecall-menu.css

@@ -46,10 +46,6 @@
     color: white;
 }
 
-#selection-menu:hover li {
-    border-color: #CCCCCC;
-}
-
 #selection-menu:after {
     content: '';
     position: absolute;

demos/iDoRecall.html

@@ -45,13 +45,6 @@
       container: document.getElementById('article'),
       id: 'selection-menu',
       menuHTML: document.querySelector('#mymenu').innerHTML,
-      onselect: function (e) {
-        var truncatedText = this.selectedText.replace(/\s+/g, ' ');
-        if (truncatedText.length > 30)
-          truncatedText = truncatedText.slice(0, 30) + '...';
-
-        this.menuHTML = this.menuHTML.replace(/ search for .*/g, ' search for <i>' + truncatedText + '</i></a>');
-      },
       handler: function (e) {
         var target = e.target,
           id = target.id,

src/selection-menu.js

@@ -103,10 +103,8 @@
             // Call the onselect handler to give it a chance to modify the menu
             instance.onselect && instance.onselect.call(instance, e);
 
-            // Fill the menu span
-            var range = window.getSelection().getRangeAt(0);
-
             // Get the start and end nodes of the selection
+            var range = window.getSelection().getRangeAt(0);
             var startNode = range.startContainer;
             var endNode = range.endContainer;
 
@@ -142,7 +140,7 @@
             // Fill the menu span
             span.innerHTML = instance.menuHTML;
 
-            // Inject the span element into the new Range
+            // Inject the span element at the start of the new Range
             newRange.insertNode(span);
 
             // Menu positioning