Skip to content

TT-7376 audio editing zoom and position fixes#339

Merged
gtryus merged 1 commit into
developfrom
TT-7376
May 29, 2026
Merged

TT-7376 audio editing zoom and position fixes#339
gtryus merged 1 commit into
developfrom
TT-7376

Conversation

@sarahentzel
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes waveform UX regressions during audio editing by preserving the user’s current playhead position and zoom level across edit-triggered reloads, so the waveform doesn’t jump back to the start or reset to fit-to-width.

Changes:

  • Restore the current playhead position after undo by reloading the audio at the current progress time.
  • Prevent automatic zoom “snap back” to fit-to-width after snip/undo by preserving and re-applying the prior zoom.
  • Adjust zoom-min synchronization so fillPx changes only raise zoom when the current zoom is below the new minimum.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/renderer/src/crud/useWaveSurfer.tsx Undo reload now restores the current position (via progress()), relying on existing clamping behavior in wsGoto.
src/renderer/src/components/WSAudioPlayerZoom.tsx Keeps user zoom when fillPx changes unless current zoom falls below the new minimum.
src/renderer/src/components/WSAudioPlayer.tsx Preserves zoom across edit reloads (snip/undo) and re-applies it on WS ready when appropriate.

@sarahentzel sarahentzel requested a review from gtryus May 29, 2026 20:03
@gtryus gtryus merged commit f84b534 into develop May 29, 2026
3 checks passed
@gtryus gtryus deleted the TT-7376 branch May 29, 2026 21:16
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.

3 participants