refactor: const for _inspInterval in inspector.html

Last remaining var declaration; matches the pattern in index.html and links.html. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
refactor: replace inline onclick with data-action event delegation
2026-05-10 23:45:42 -04:00 · 2026-05-10 23:45:09 -04:00
2 changed files with 6 additions and 5 deletions
@@ -144,9 +144,9 @@
      <!-- ⌘K affordance -->
      <button type="button"
              class="lt-btn lt-btn-ghost lt-btn-sm lt-cmd-hint-btn"
+              data-action="open-cmdpalette"
              title="Command palette (Ctrl+K)"
-              aria-label="Open command palette"
-              onclick="if(window.lt&&lt.cmdPalette)lt.cmdPalette.open()">&#x2315;&nbsp;K</button>
+              aria-label="Open command palette">&#x2315;&nbsp;K</button>

      <button type="button" class="lt-theme-btn" id="lt-theme-btn"
              aria-label="Toggle theme" title="Toggle light/dark mode">&#x2600;</button>
@@ -346,8 +346,9 @@
      const btn = e.target.closest('[data-action]');
      if (!btn) return;
      const action = btn.getAttribute('data-action');
-      if (action === 'show-keyboard-help' && window.lt)  lt.modal.open('lt-keys-help');
-      if (action === 'open-settings' && window.lt)       lt.modal.open('lt-settings-modal');
+      if (action === 'open-cmdpalette' && window.lt && lt.cmdPalette) lt.cmdPalette.open();
+      if (action === 'show-keyboard-help' && window.lt)               lt.modal.open('lt-keys-help');
+      if (action === 'open-settings' && window.lt)                    lt.modal.open('lt-settings-modal');
    });

    lt.keys.on('r', function() { lt.autoRefresh.now(); });
@@ -468,7 +468,7 @@ async function loadInspector() {
 }

 loadInspector();
-var _inspInterval = (window.gandalfSettings && window.gandalfSettings.refreshInterval) || 60;
+const _inspInterval = (window.gandalfSettings && window.gandalfSettings.refreshInterval) || 60;
 if (_inspInterval > 0) lt.autoRefresh.start(loadInspector, Math.max(_inspInterval, 15) * 1000);

 window.onGandalfSettingsChanged = function(s) {
Author	SHA1	Message	Date
jared	0f2506d5a4	refactor: const for _inspInterval in inspector.html Lint / Python (flake8) (push) Successful in 54s Details Lint / JS (eslint) (push) Successful in 9s Details Security / Python Security (bandit) (push) Successful in 1m17s Details Test / Python Tests (pytest) (push) Successful in 53s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 3s Details Last remaining var declaration; matches the pattern in index.html and links.html. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-10 23:45:42 -04:00
jared	678ede4e76	refactor: replace inline onclick with data-action event delegation Lint / Python (flake8) (push) Successful in 42s Details Lint / JS (eslint) (push) Successful in 8s Details Security / Python Security (bandit) (push) Successful in 1m0s Details Test / Python Tests (pytest) (push) Successful in 50s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 2s Details The command palette button used an inline onclick handler while every other interactive element in base.html uses data-action + event delegation. Now consistent: data-action="open-cmdpalette" handled in the global footer click listener. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-10 23:45:09 -04:00