ryan/games
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 9 Wiki Activity

fix: use templ.NewOnceHandle to prevent script duplication on SSE patches
All checks were successful
CI / Deploy / test (pull_request) Successful in 15s
Details
CI / Deploy / lint (pull_request) Successful in 25s
Details
CI / Deploy / deploy (pull_request) Has been skipped
Details

Browse Source 
Replace ConnectionIndicatorWithScript wrapper with a single ConnectionIndicator
component that uses templ.NewOnceHandle() to ensure the watcher script is only
rendered once per page, even when the indicator is patched via SSE.
This commit is contained in:
Ryan Hamamura
2026-03-03 10:39:26 -10:00
parent 589d1f09e8
commit d6e64763cc
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
features/common/components/shared.templ
View File

@@ -52,6 +52,8 @@ func isStale(lastPing int64) bool {
return lastPing == 0 return lastPing == 0
} }
var connectionWatcherHandle = templ.NewOnceHandle()
// ConnectionIndicator shows a small dot indicating SSE connection status. // ConnectionIndicator shows a small dot indicating SSE connection status.
// Server patches this with a timestamp; client JS detects staleness. // Server patches this with a timestamp; client JS detects staleness.
templ ConnectionIndicator(lastPing int64) { templ ConnectionIndicator(lastPing int64) {
@@ -79,7 +81,9 @@ templ ConnectionIndicator(lastPing int64) {
></div> ></div>
</div> </div>
</div> </div>
@connectionWatcher() @connectionWatcherHandle.Once() {
@connectionWatcher()
}
} }
script connectionWatcher() { script connectionWatcher() {