Fix contest panel keyboard accessibility#1037
Merged
accius merged 9 commits intoMay 31, 2026
Merged
Conversation
Release v26.3.1 — Staging → main
Hotfix: guard PropagationPanel against empty/invalid timeZone (RangeError on first load)
…g_line787 [Hotfix main] bug fix spelling of console.debug
Release v26.3.2 — pre-Hamvention drop (MeshCom + VOACAP hotfix)
whatsnew(26.3.2): correct MeshCom contributor attribution
Cuts Staging into main for the v26.3.3 release.
…ction
The socket idle timeout (60s) was the shortest of all failure timers, so
any 60s gap between spots — normal on quiet bands overnight — tore down a
healthy connection. Keepalive (120s) was too slow to prevent it and the
180s activity watchdog (the graceful node-failover) never got to run.
- socket timeout 60s -> 300s, as a last-resort TCP backstop; the 180s
activity watchdog now owns failover as designed
- keepalive 120s -> 60s so the connection stays warm
- destroy the socket in the 'timeout' handler — Node does not auto-close
on timeout, leaving a zombie socket that kept feeding data and
spuriously reset the failover counter after [RECONNECT]
- mark authenticated on first spot; the DXSpider prompt has no trailing
newline so prompt-based detection never matched, and sh/dx output lines
("...de Helmut<DF4IY>") false-matched the prompt regex
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
Author
|
Contributor
Author
|
Screenshots show the WA7BNM link toggle enabled/disabled in the Contest Panel. When enabled, contest names can be opened, show the clickable styling, and turn blue on mouse hover. When disabled, contest names no longer act as links. The visible focus outline on the 🔗 button is expected and confirms the control is keyboard-focusable. |
Owner
|
quick note, this PR was targeting our main production branch and all PRs should target Staging. I have updated the target branch and am looking at the diff now. will follow up with notes if there are any. |
This was referenced May 31, 2026
Contributor
Author
|
Sorry about that — I recreated the branch and the history got messier than intended. Thanks for cleaning it up. I’ll make sure future PRs are branched from Staging and that the final diff only includes the intended changes. |
12 tasks
accius
added a commit
that referenced
this pull request
Jun 2, 2026
Four people have been carrying the day-to-day with Chris this cycle — calling them out as core maintainers in both the WhatsNew banner and a new Core Maintainers block above the existing contributors pill list in Settings → Community. - Jörg Holzapfel (DO1HOZ / @ceotjoe) — rig-bridge, Cloud Relay, MeshCom - Alan Hargreaves (@alanhargreaves) — server reliability, local install - Laura Batalha (@lbatalha) — API design, Docker / GHCR infrastructure - Michael R Wheeley (@MichaelWheeley) — satellites, geo utils, i18n Also adds JoshuaLutgendorff (first-time contributor PR #1037) to the contributors pill list and removes ceotjoe + alanhargreaves from it since they're now in the maintainers block above.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Fixes keyboard accessibility in
ContestPanelby replacing clickable<span>elements with real<button>controls for the WA7BNM link toggle and contest names.The WA7BNM toggle now exposes button semantics, an accessible label, and
aria-pressed. Contest names now render as keyboard-accessible buttons with accessible labels when contest rows are available.Closes #1030
Type of change
How to test
npm run build.npm run test:run.npm run dev, tab to the WA7BNM link toggle, and confirm it is focusable and toggles on/off with the keyboard.Checklist
server.js: caches have TTLs and size caps (we serve 2,000+ concurrent users)var(--accent-cyan), etc.).bak,.old,console.logdebug lines, or test scripts includedScreenshots (if visual change)