Fix #111: Recording Delete Button in wrong position #113

Closed
Tasch wants to merge 1 commit from batch/111 into master
Owner
No description provided.
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
4a96a16558
The delete button was previously either:
- A raw action-row outside any section-toggle (!hasTranscript case)
- Inside the transcript section-content (hasTranscript case)
- Inside playSection which had no corresponding toggle button

Now each case has the delete button in its own '🗑️ Delete Recording'
section-toggle with a proper section-content wrapper, consistent
with the structured transcript section pattern.
Tasch self-assigned this 2026-04-26 12:18:30 +00:00
ui/index.html Outdated
@ -1910,4 +1934,3 @@
</button>
<div class="section-content" id="transcript-${escapeHtml(folder.name)}" style="display:none">
<div class="action-row">
<button class="action-btn delete" title="Delete" onclick="deleteFolder('${escapeHtml(folder.name)}')">🗑️</button>
Owner

NO! YOU DELETED THE WRONG ITEM!!!!!

NO! YOU DELETED THE WRONG ITEM!!!!!
ui/index.html Outdated
@ -1787,4 +1800,3 @@
</button>
<div class="section-content" id="cur-transcript" style="display:none">
<div class="action-row">
<button class="action-btn delete" title="Delete recording" onclick="deleteFolder('${escapeHtml(session.folderName)}')">🗑️</button>
Owner

NO! YOU DELETED THE WRONG ITEM!!!!!

NO! YOU DELETED THE WRONG ITEM!!!!!
ui/index.html Outdated
@ -1774,3 +1774,2 @@
</button>
<div class="action-row">
<button class="action-btn delete" title="Delete" onclick="deleteFolder('${escapeHtml(session.folderName)}')">🗑️</button>
<button class="section-toggle" onclick="toggleSection(this, 'cur-recording-actions')">
Owner

WRONG! KEEP "


" BUT IN CORRECT LOCATION!!!!!

WRONG! KEEP "<div class="action-row"> <button class="action-btn delete" title="Delete" onclick="deleteFolder('${escapeHtml(session.folderName)}')">🗑️</button>" BUT IN CORRECT LOCATION!!!!!
ui/index.html Outdated
@ -1896,6 +1904,14 @@
<button class="section-toggle transcribe" onclick="playRecording('${escapeHtml(folder.name)}')">
▶️ Play Recording
</button>
<button class="section-toggle" onclick="toggleSection(this, 'recording-actions-${escapeHtml(folder.name)}')">
Owner

WHY YOU DID THIS!?!?!? ONLY MOVE THE "


" TO CORRECT LOCATION!

WHY YOU DID THIS!?!?!? ONLY MOVE THE " <div class="action-row"> <button class="action-btn delete" title="Delete" onclick="deleteFolder('${escapeHtml(session.folderName)}')">🗑️</button>" TO CORRECT LOCATION!
Tasch force-pushed batch/111 from 4a96a16558
Some checks are pending
CI / lint (pull_request) Waiting to run
CI / test (pull_request) Waiting to run
to 5484565f8c
Some checks failed
CI / lint (pull_request) Has been cancelled
CI / test (pull_request) Has been cancelled
2026-04-26 12:35:58 +00:00
Compare
SaschaFuksa closed this pull request 2026-04-26 12:54:16 +00:00
Some checks are pending
CI / lint (pull_request) Has been cancelled
CI / test (pull_request) Has been cancelled
ci/lint
Required
ci/test
Required

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
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!113
No description provided.