From a38fecd20b43b2217d4b7b6af1de02948ddf19e5 Mon Sep 17 00:00:00 2001
From: Rob Pilling <robpilling@gmail.com>
Date: Thu, 17 Apr 2025 22:34:16 +0100
Subject: [PATCH] runplus: only reacquire layout UI if recorder might've
 rendered

timings for start/stop:
13:38:15.996Z: onStartStop()
13:38:16.006Z: setRecording()
13:38:16.728Z: setRecording() done
13:38:16.732Z: layout.render()
13:38:17.154Z: layout.render() done
13:38:17.157Z: exs.start/stop/resume()
13:38:17.161Z: exs.start/stop/resume() done
13:38:17.462Z: setStatus()

13:38:19.027Z: onStartStop()
13:38:19.199Z: exs.start/stop/resume()
13:38:19.202Z: exs.start/stop/resume() done
13:38:19.495Z: setStatus()

13:38:24.795Z: onStartStop()
13:38:24.805Z: setRecording()
13:38:25.483Z: setRecording() done
13:38:25.487Z: layout.render()
13:38:25.913Z: layout.render() done
13:38:25.915Z: exs.start/stop/resume()
13:38:25.919Z: exs.start/stop/resume() done
13:38:26.218Z: setStatus()

13:38:27.842Z: onStartStop()
13:38:28.010Z: exs.start/stop/resume()
13:38:28.012Z: exs.start/stop/resume() done
13:38:28.309Z: setStatus()

in summary:
setRecording: ~700ms
layout.render(): ~420ms
---
 apps/runplus/app.js | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/apps/runplus/app.js b/apps/runplus/app.js
index 93b341a51..e15a4bd34 100644
--- a/apps/runplus/app.js
+++ b/apps/runplus/app.js
@@ -92,9 +92,12 @@ function onStartStop() {
         then(() => WIDGETS["recorder"].setRecording(true, { force : shouldResume?"append":undefined })).
         then(() => {
           screen = "main";
-          layout.setUI(); // grab our input handling again
-          layout.forgetLazyState();
-          layout.render();
+          if(!shouldResume){
+            // setRecording might have rendered - need to grab UI
+            layout.setUI(); // grab our input handling again
+            layout.forgetLazyState();
+            layout.render();
+          }
         });
     } else {
       promise = promise.then(