From: Scott Worley <scottworley@scottworley.com>
Date: Wed, 2 Mar 2022 23:54:00 +0000 (-0800)
Subject: 'l' goes to first *visible* subtask
X-Git-Url: http://git.scottworley.com/vopamoi/commitdiff_plain/f36b20d6c8aff57f2582ffc800d20f03ab0bb44c?ds=sidebyside

'l' goes to first *visible* subtask

I.e., it does something even if the first subtask is hidden because it's
complete or something.
---

diff --git a/vopamoi.ts b/vopamoi.ts
index 5f759d0..8e72809 100644
--- a/vopamoi.ts
+++ b/vopamoi.ts
@@ -378,8 +378,9 @@ function BrowserUI() {
       return target as HTMLElement;
     },
 
-    firstVisibleTask: function () {
-      for (const task of document.getElementsByClassName("task")) {
+    firstVisibleTask: function (root: Element | null = null) {
+      if (root === null) root = document.body;
+      for (const task of root.getElementsByClassName("task")) {
         const state = task.getAttribute("data-state");
         if (
           task instanceof HTMLElement &&
@@ -447,7 +448,9 @@ function BrowserUI() {
     },
 
     moveCursorRight: function () {
-      (this.currentTask()?.getElementsByClassName("task")[0] as HTMLElement)?.focus();
+      const active = this.currentTask();
+      if (!active) return false;
+      (this.firstVisibleTask(active) as HTMLElement | null)?.focus();
     },
 
     moveCursorVertically: function (offset: number): boolean {