From 64037c08321bbabda36fdbbfe03e132a3265a129 Mon Sep 17 00:00:00 2001
From: "Joshua A. Horton" <joshua_horton@sil.org>
Date: Thu, 22 Sep 2022 10:31:01 +0700
Subject: [PATCH] feat(web): content editable editing for touch

---
 web/source/dom/domManager.ts          | 4 +---
 web/testing/attachment-api/index.html | 4 +---
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/web/source/dom/domManager.ts b/web/source/dom/domManager.ts
index f47a0e0cc2..d3f8c3ddc7 100644
--- a/web/source/dom/domManager.ts
+++ b/web/source/dom/domManager.ts
@@ -522,7 +522,7 @@ namespace com.keyman.dom {
           console.warn("Error during attachment to / detachment from iframe: ");
           console.warn(err);
         }
-      } else if(x.isContentEditable && !touchable) { // Only allow contentEditable attachment outside of 'touch' mode.
+      } else if(x.isContentEditable) {
         return true;
       }
 
@@ -775,8 +775,6 @@ namespace com.keyman.dom {
           this.disableTouchElement(Pelem);
           this.setupNonKMWTouchElement(Pelem);
 
-          var keyman = this.keyman;
-
           // If a touch alias was removed, chances are it's gonna mess up our touch-based layout scheme, so let's update the touch elements.
           window.setTimeout(function() {
             this.listInputs();
diff --git a/web/testing/attachment-api/index.html b/web/testing/attachment-api/index.html
index c02c66d3fd..69fb497c15 100644
--- a/web/testing/attachment-api/index.html
+++ b/web/testing/attachment-api/index.html
@@ -93,9 +93,7 @@
 			<p><em>Note:</em> The iframe section should not actually attach/enable for touch devices
 			and does not support <code>setKeyboardForControl</code>.</p>
 		</div>
-		<div id='DynamicEditables'><h3><code>contenteditable</code> Elements:</h3>
-			<p><em>Note:</em> This section should not actually attach/enable for touch devices.</p>
-		</div>
+		<div id='DynamicEditables'><h3><code>contenteditable</code> Elements:</h3></div>
   </div>
 	<hr/>
    <h3><a href="../index.html">Return to testing home page</a></h3>