From 573453aad0d0ee0193ae2e8110f883a43f6179ec Mon Sep 17 00:00:00 2001
From: Richard Livsey <richard@livsey.org>
Date: Thu, 3 Dec 2015 00:08:06 +0000
Subject: [PATCH] Atoms should not be editable

---
 src/js/renderers/editor-dom.js          | 1 +
 tests/unit/renderers/editor-dom-test.js | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/js/renderers/editor-dom.js b/src/js/renderers/editor-dom.js
index 44ab4b1ec..9062f1bef 100644
--- a/src/js/renderers/editor-dom.js
+++ b/src/js/renderers/editor-dom.js
@@ -113,6 +113,7 @@ function renderCard() {
 
 function renderAtom(element, previousRenderNode) {
   let atomElement = document.createElement('span');
+  atomElement.contentEditable = false;
   addClassName(atomElement, '-mobiledoc-kit__atom');
 
   if (previousRenderNode) {
diff --git a/tests/unit/renderers/editor-dom-test.js b/tests/unit/renderers/editor-dom-test.js
index eaa7ee046..66cf05fb3 100644
--- a/tests/unit/renderers/editor-dom-test.js
+++ b/tests/unit/renderers/editor-dom-test.js
@@ -210,7 +210,7 @@ test('renders a post with atom', (assert) => {
       }
     }
   ]);
-  assert.equal(renderTree.rootElement.innerHTML, `<p><span class="-mobiledoc-kit__atom">@bob</span></p>`);
+  assert.equal(renderTree.rootElement.innerHTML, `<p><span contenteditable="false" class="-mobiledoc-kit__atom">@bob</span></p>`);
 });
 
 test('renders a post with mixed markups and atoms', (assert) => {
@@ -238,7 +238,7 @@ test('renders a post with mixed markups and atoms', (assert) => {
     }
   ]);
 
-  assert.equal(renderTree.rootElement.innerHTML, `<p><b>bold<i>italic <span class="-mobiledoc-kit__atom">@bob</span></i> bold</b>text.</p>`);
+  assert.equal(renderTree.rootElement.innerHTML, `<p><b>bold<i>italic <span contenteditable="false" class="-mobiledoc-kit__atom">@bob</span></i> bold</b>text.</p>`);
 });
 
 test('renders a card section', (assert) => {