Fix #111: Move recording Delete button into its own section-toggle #116

Merged
SaschaFuksa merged 2 commits from batch/111-v2 into master 2026-04-26 12:54:10 +00:00
Owner

Fix #111 — Recording Delete Button Position

Problem

The Delete button for recordings was positioned after the section-toggle buttons but NOT inside a collapsible section. It appeared permanently visible, not under its own section-toggle.

Solution

Current Recording (no transcript):

  • ▶ Run Recording + 🗑️ Delete (collapsible) — Delete = deleteFolder()

Current Recording (has transcript):

  • ▶️ Play Recording + 🗑️ Delete (collapsible) + 📄 Show Transcript
  • Transcript section: 📝✕ Delete transcript + ↻ Recreate — Delete = deleteTranscriptOnly()

Previous Recordings (has transcript):

  • ▶️ Play Recording + 🗑️ Delete (collapsible) + 📄 Show Transcript
  • Transcript section: 📝✕ Delete transcript + ↻ Recreate

Previous Recordings (no transcript):

  • ▶️ Play Recording + 🗑️ Delete (collapsible) + ▶ Run Recording

Key changes

  • 🗑️ Delete is now a section-toggle with toggleSection() — opens section-content with deleteFolder() button
  • 📝✕ Delete transcript stays inside the transcript section — calls deleteTranscriptOnly()
  • playSection (shown during playback) is now empty — no duplicate delete button inside it

Files changed

  • ui/index.html — 26 insertions, 9 deletions
## Fix #111 — Recording Delete Button Position ### Problem The Delete button for recordings was positioned after the section-toggle buttons but NOT inside a collapsible section. It appeared permanently visible, not under its own section-toggle. ### Solution **Current Recording (no transcript):** - `▶ Run Recording` + `🗑️ Delete` (collapsible) — Delete = `deleteFolder()` **Current Recording (has transcript):** - `▶️ Play Recording` + `🗑️ Delete` (collapsible) + `📄 Show Transcript` - Transcript section: `📝✕ Delete transcript` + `↻ Recreate` — Delete = `deleteTranscriptOnly()` **Previous Recordings (has transcript):** - `▶️ Play Recording` + `🗑️ Delete` (collapsible) + `📄 Show Transcript` - Transcript section: `📝✕ Delete transcript` + `↻ Recreate` **Previous Recordings (no transcript):** - `▶️ Play Recording` + `🗑️ Delete` (collapsible) + `▶ Run Recording` ### Key changes - `🗑️ Delete` is now a `section-toggle` with `toggleSection()` — opens `section-content` with `deleteFolder()` button - `📝✕ Delete transcript` stays inside the transcript section — calls `deleteTranscriptOnly()` - `playSection` (shown during playback) is now empty — no duplicate delete button inside it ### Files changed - `ui/index.html` — 26 insertions, 9 deletions
Fix #111: Move recording Delete button into its own section-toggle
Some checks are pending
CI / lint (pull_request) Waiting to run
CI / test (pull_request) Waiting to run
4db0986e57
Current recording (no transcript):
- Run Recording + Delete (collapsible)

Current recording (has transcript):
- Play Recording + Delete (collapsible) + Show Transcript
- Transcript section: Delete transcript + Recreate (NOT Delete recording)

Previous recordings:
- Play Recording + Delete (collapsible) + Show Transcript
- Transcript section: Delete transcript + Recreate
- playSection (shown while playing) is now empty — no duplicate delete button

Delete recording = deleteFolder() (always)
Delete transcript = deleteTranscriptOnly() (only in transcript section)
Fix #111, #114, #115: Delete section-toggle, timer fix, Run→Transcript
Some checks failed
CI / lint (pull_request) Has been cancelled
CI / test (pull_request) Has been cancelled
590073b0ef
Issues fixed:
- #111: Delete button now a section-toggle (collapsible), not permanently visible
- #114: 'Run Recording' → '▶ Transcript' for current session (no transcript yet)
- #115: Timer now stops when recording stops (clearInterval in stopSession)

Also removed:
- 📝✕ Delete transcript button (was incorrectly added, user wants it gone)
- playSection now empty (no duplicate delete inside it)
SaschaFuksa deleted branch batch/111-v2 2026-04-26 12:54:10 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Tasch/whisper-transcribe!116
No description provided.