Making the bold functionality better

Author: andrerpena

src/ReactMdeCommands.js

@@ -1,4 +1,4 @@
-import { insertText } from './TextHelper';
+import { insertText, getSurroundingWord } from './TextHelper';
 
 export default {
 
@@ -10,6 +10,13 @@ export default {
      * @returns
      */
     makeBold: function (text, selection) {
+
+        if (selection[0] == selection[1]) {
+            // the user is pointing to a word
+            selection = getSurroundingWord(text, selection[0]).position;
+
+        }
+        // the user is selecting a word section
         var newText = insertText(text, '**', selection[0]);
         newText = insertText(newText, '**', selection[1] + 2);
         return {

src/TextHelper.js

@@ -44,7 +44,6 @@ export function getSurroundingWord(text, position) {
 
     return {
         word: text.slice(leftIndex, rightIndex),
-        leftIndex: leftIndex,
-        rightIndex: rightIndex
+        position: [leftIndex,rightIndex]
     };
 }

test/TextHelperSpec.js

@@ -27,80 +27,80 @@ describe("TextHelperSpec.js", function () {
         it('Basic usage', function () {
             var result = getSurroundingWord('bob went to school', 5);
             assert.strictEqual(result.word, 'went');
-            assert.strictEqual(result.leftIndex, 4);
-            assert.strictEqual(result.rightIndex, 8);
+            assert.strictEqual(result.position[0], 4);
+            assert.strictEqual(result.position[1], 8);
         });
 
         it('Word edge to the right', function () {
             var result = getSurroundingWord('bob went to school', 8);
             assert.strictEqual(result.word, 'went');
-            assert.strictEqual(result.leftIndex, 4);
-            assert.strictEqual(result.rightIndex, 8);
+            assert.strictEqual(result.position[0], 4);
+            assert.strictEqual(result.position[1], 8);
         });
 
         it('Word edge to the left', function () {
             var result = getSurroundingWord('bob went to school', 4);
             assert.strictEqual(result.word, 'went');
-            assert.strictEqual(result.leftIndex, 4);
-            assert.strictEqual(result.rightIndex, 8);
+            assert.strictEqual(result.position[0], 4);
+            assert.strictEqual(result.position[1], 8);
         });
 
         it('text beginning', function () {
             var result = getSurroundingWord('bob went to school', 0);
             assert.strictEqual(result.word, 'bob');
-            assert.strictEqual(result.leftIndex, 0);
-            assert.strictEqual(result.rightIndex, 3);
+            assert.strictEqual(result.position[0], 0);
+            assert.strictEqual(result.position[1], 3);
         });
 
         it('text ending', function () {
             var result = getSurroundingWord('bob went to school', 18);
             assert.strictEqual(result.word, 'school');
-            assert.strictEqual(result.leftIndex, 12);
-            assert.strictEqual(result.rightIndex, 18);
+            assert.strictEqual(result.position[0], 12);
+            assert.strictEqual(result.position[1], 18);
         });
 
         // tests with line breaks
         it('Basic usage with line breaks', function () {
             var result = getSurroundingWord('bob\nwent\nto school', 5);
             assert.strictEqual(result.word, 'went');
-            assert.strictEqual(result.leftIndex, 4);
-            assert.strictEqual(result.rightIndex, 8);
+            assert.strictEqual(result.position[0], 4);
+            assert.strictEqual(result.position[1], 8);
         });
 
         it('text ending with line break', function () {
             var result = getSurroundingWord('bob went to school\n', 18);
             assert.strictEqual(result.word, 'school');
-            assert.strictEqual(result.leftIndex, 12);
-            assert.strictEqual(result.rightIndex, 18);
+            assert.strictEqual(result.position[0], 12);
+            assert.strictEqual(result.position[1], 18);
         });
 
         it('text beginning with line break', function () {
             var result = getSurroundingWord('\nbob went to school', 1);
             assert.strictEqual(result.word, 'bob');
-            assert.strictEqual(result.leftIndex, 1);
-            assert.strictEqual(result.rightIndex, 4);
+            assert.strictEqual(result.position[0], 1);
+            assert.strictEqual(result.position[1], 4);
         });
 
         // edge cases
         it('within spaces', function() {
             var result = getSurroundingWord('   ', 1);
             assert.strictEqual(result.word, '');
-            assert.strictEqual(result.leftIndex, 1);
-            assert.strictEqual(result.rightIndex, 1);
+            assert.strictEqual(result.position[0], 1);
+            assert.strictEqual(result.position[1], 1);
         });
 
         it('within spaces 2', function() {
             var result = getSurroundingWord('   ', 2);
             assert.strictEqual(result.word, '');
-            assert.strictEqual(result.leftIndex, 2);
-            assert.strictEqual(result.rightIndex, 2);
+            assert.strictEqual(result.position[0], 2);
+            assert.strictEqual(result.position[1], 2);
         });
 
         it('within line-breaks', function() {
             var result = getSurroundingWord('\n\n\n', 1);
             assert.strictEqual(result.word, '');
-            assert.strictEqual(result.leftIndex, 1);
-            assert.strictEqual(result.rightIndex, 1);
+            assert.strictEqual(result.position[0], 1);
+            assert.strictEqual(result.position[1], 1);
         });
     });