From bcf73844e05126e17ca1174fc2440f5837b7b483 Mon Sep 17 00:00:00 2001
From: Todd Burnside <tburnside@gemini.edu>
Date: Wed, 18 Dec 2024 12:07:10 -0800
Subject: [PATCH] Use a ref for TargetName input instead of direct dom lookup

---
 .../explore/targeteditor/SearchForm.scala     | 28 ++++++++-----------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/explore/src/main/scala/explore/targeteditor/SearchForm.scala b/explore/src/main/scala/explore/targeteditor/SearchForm.scala
index 0fc1c6c11e..a3c860a910 100644
--- a/explore/src/main/scala/explore/targeteditor/SearchForm.scala
+++ b/explore/src/main/scala/explore/targeteditor/SearchForm.scala
@@ -29,7 +29,6 @@ import lucuma.ui.primereact.given
 import lucuma.ui.reusability.given
 import lucuma.ui.syntax.all.given
 import org.scalajs.dom
-import org.scalajs.dom.HTMLButtonElement
 
 import scalajs.js.timers
 import scalajs.js.JSConverters.*
@@ -55,22 +54,19 @@ object SearchForm:
       .useEffectWithDepsBy((props, _, _, _, _) => props.targetName.get)((_, _, term, enabled, _) =>
         name => term.set(name) >> enabled.setState(true)
       )
-      .useEffectWithDepsBy((props, _, _, _, _) => props.id)((props, _, _, _, _) =>
-        _ => // Auto-select name field on new targets.
-          // Accessing dom elements by id is not ideal in React, but passing a ref to the <input> element of a
-          // Form.Input is a bit convoluted. We should reevaluate when and if we switch to another component library.
-          // TODO REEVALUATE NOW!
-          CallbackTo(
-            Option(dom.document.getElementById("search"))
-              .foldMap(node =>
-                Callback(timers.setTimeout(0)(node.asInstanceOf[dom.HTMLInputElement].select()))
+      .useRefToVdom[dom.HTMLButtonElement]
+      .useRefToVdom[dom.HTMLInputElement]
+      .useEffectWithDepsBy((props, _, _, _, _, _, inputRef) => (props.id, inputRef))(
+        (props, _, _, _, _, _, _) =>
+          // Auto select name field on new targets
+          (_, inputRef) =>
+            inputRef.get
+              .flatMap(
+                _.foldMap(i => Callback(timers.setTimeout(50)(i.select())))
               )
-          ).flatten
-            .when(props.targetName.get === NewTargetName)
-            .void
+              .when_(props.targetName.get === NewTargetName)
       )
-      .useRefToVdom[HTMLButtonElement]
-      .render: (props, ctx, term, enabled, error, buttonRef) =>
+      .render: (props, ctx, term, enabled, error, buttonRef, inputRef) =>
         import ctx.given
 
         val searchComplete: Callback = props.searching.mod(_ - props.id)
@@ -123,4 +119,4 @@ object SearchForm:
           onTextChange = (_: String) => error.setState(none),
           onValidChange = valid => enabled.setState(valid),
           placeholder = "Name"
-        ).withMods(^.onKeyPress ==> onKeyPress)
+        ).withMods(^.onKeyPress ==> onKeyPress, ^.untypedRef := inputRef)