Skip to content

feat(#209): per-cue-point enable/disable toggle#244

Open
Radexito wants to merge 3 commits into
devfrom
feat/209-cue-point-enable-toggle
Open

feat(#209): per-cue-point enable/disable toggle#244
Radexito wants to merge 3 commits into
devfrom
feat/209-cue-point-enable-toggle

Conversation

@Radexito

Copy link
Copy Markdown
Owner

Summary

  • DB: enabled INTEGER NOT NULL DEFAULT 1 column added to cue_points via safe migration — existing rows default to enabled
  • CuePointsEditor: each row now has a ●/○ toggle button; disabled rows are dimmed (0.35 opacity)
  • PlayerBar: disabled cues are hidden from the seekbar cue markers
  • Rekordbox export: disabled cues excluded from PCOB, extPCOB, and PCO2 sections in ANLZ files

Test plan

  • Open cue editor — all cues show ● (enabled)
  • Click ○ to disable — row dims, marker disappears from seekbar
  • Export to USB — disabled cues absent from Rekordbox ANLZ
  • Re-enable — marker reappears, export includes cue again

Closes #209

🤖 Generated with Claude Code

- DB: add enabled INTEGER NOT NULL DEFAULT 1 column to cue_points via
  migration (existing rows default to enabled)
- cuePointRepository: accept enabled field in updateCuePoint
- main.js: forward enabled field from update-cue-point IPC handler
- CuePointsEditor: toggle button (●/○) per row; disabled rows are dimmed
- PlayerBar: skip disabled cue markers on the seekbar
- anlzWriter: exclude disabled cues from PCOB, extPCOB, and PCO2 sections
  on Rekordbox USB export

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Radexito Radexito force-pushed the feat/209-cue-point-enable-toggle branch from 9b2a91c to b83c2e8 Compare April 18, 2026 11:34
Radexito and others added 2 commits April 19, 2026 15:25
…icts

Takes dev's export-toggle naming (⊘/⊙ icons, USB export tooltip) and
simpler migrations.js try-catch over the older HEAD variants.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.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