Skip to content

fix: top alert when save and delete behaviour#99

Merged
destucr merged 1 commit intodevelopmentfrom
fix/alert-notification
Nov 30, 2025
Merged

fix: top alert when save and delete behaviour#99
destucr merged 1 commit intodevelopmentfrom
fix/alert-notification

Conversation

@destucr
Copy link
Contributor

@destucr destucr commented Nov 30, 2025

🎯 Summary

This pull request introduces major UI and animation enhancements to the success alerts in both the Live Listening (OrbLiveListenView) and Saved Recording Playback (SavedRecordingPlaybackView) views. The goal is to provide a cleaner, smoother, and more visually consistent user experience, especially around save and delete actions.


✨ Features/Changes

🔊 Live Listening View (OrbLiveListenView)

  • Cleaner Alert Animation: The success alert now uses a slide-down transition from the top combined with an opacity change, replacing the previous dark overlay for a more modern appearance.
  • Reduced Padding: Adjusted the success alert's vertical padding for improved visual placement.
  • Coordinated Control Hiding: Controls (like topControlsView, statusTextView, and the floating library button) now fade out smoothly and are kept hidden whenever the success alert is visible.

🗑️ Saved Playback View (SavedRecordingPlaybackView)

  • Consistent Control Hiding: Implemented matching logic to ensure topControlsView, nameAndDateView, and statusTextView fade out when the delete success alert is shown, maintaining visual harmony with the live view.
  • Smoother Post-Delete Flow: Introduced a slight delay between the dismissal of the success alert and the reappearance of the timeline view, providing a more deliberate and less jarring transition after a recording is deleted.

📁 Files Changed

  • OrbLiveListenView.swift - Updated success alert presentation logic, removed overlay, and added control fade animations.
  • SavedRecordingPlaybackView.swift - Implemented consistent control hiding for delete alert and refined post-delete transition timing.

🔧 Usage Examples (if applicable)

// N/A: Changes are primarily UI/animation updates within existing view logic.

@destucr destucr merged commit ed93bba into development Nov 30, 2025
2 checks passed
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