Skip to content

TUI: wallpaper source constants and preview debounce clarity - #46

Merged
Skeyelab merged 1 commit into
mainfrom
feat/tui-readability
May 4, 2026
Merged

Skeyelab merged 1 commit into
mainfrom
feat/tui-readability

Conversation

@Skeyelab

@Skeyelab Skeyelab commented May 4, 2026

Copy link
Copy Markdown
Member

Summary

Readability pass on tui.go from beads review (omarchy-wled-jip, omarchy-wled-t7t, omarchy-wled-507).

  • Wallpaper row: sourceIndexAccent / Foreground / Wallpaper instead of magic 2.
  • Preview: single previewScheduleID; previewTickMsg.scheduleID — ticks compare against latest schedule (drop redundant previewArm).
  • Swatch: solid preview uses r0,g0,b0 so we do not shadow strings.Builder b.

Tests

go test ./... · go vet ./...

Beads

Refs: omarchy-wled-jip, omarchy-wled-t7t, omarchy-wled-507 (close when merged).

- Add sourceIndexAccent/Foreground/Wallpaper constants (replace magic 2)
- Use a single previewScheduleID; ticks carry scheduleID and skip if stale
- Solid preview swatch uses r0,g0,b0 without shadowing strings.Builder

Refs: omarchy-wled-jip, omarchy-wled-t7t, omarchy-wled-507
Co-authored-by: Cursor <cursoragent@cursor.com>
@Skeyelab
Skeyelab merged commit 1bb00d9 into main May 4, 2026
1 check passed
Skeyelab added a commit that referenced this pull request May 4, 2026
Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant