Display voice message progress cursor if progress is greater than zero (#2077)

This commit is contained in:
Nicolas Mauri 2023-11-13 16:41:38 +01:00 committed by GitHub
parent bc46c48b2e
commit acd079199a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
7 changed files with 14 additions and 16 deletions

View File

@ -41,7 +41,9 @@ class AudioPlayerState: ObservableObject, Identifiable {
/// updates are delayed by a fixed amount of time /// updates are delayed by a fixed amount of time
@Published private(set) var playerButtonPlaybackState: AudioPlayerPlaybackState @Published private(set) var playerButtonPlaybackState: AudioPlayerPlaybackState
@Published private(set) var progress: Double @Published private(set) var progress: Double
@Published private(set) var showProgressIndicator: Bool var showProgressIndicator: Bool {
progress > 0
}
private weak var audioPlayer: AudioPlayerProtocol? private weak var audioPlayer: AudioPlayerProtocol?
private var audioPlayerSubscription: AnyCancellable? private var audioPlayerSubscription: AnyCancellable?
@ -65,7 +67,6 @@ class AudioPlayerState: ObservableObject, Identifiable {
self.duration = duration self.duration = duration
self.waveform = waveform ?? EstimatedWaveform(data: []) self.waveform = waveform ?? EstimatedWaveform(data: [])
self.progress = progress self.progress = progress
showProgressIndicator = false
playbackState = .stopped playbackState = .stopped
playerButtonPlaybackState = .stopped playerButtonPlaybackState = .stopped
setupPlaybackStateSubscription() setupPlaybackStateSubscription()
@ -79,7 +80,6 @@ class AudioPlayerState: ObservableObject, Identifiable {
func updateState(progress: Double) async { func updateState(progress: Double) async {
let progress = max(0.0, min(progress, 1.0)) let progress = max(0.0, min(progress, 1.0))
self.progress = progress self.progress = progress
showProgressIndicator = true
if let audioPlayer { if let audioPlayer {
var shouldResumeProgressPublishing = false var shouldResumeProgressPublishing = false
if audioPlayer.state == .playing { if audioPlayer.state == .playing {
@ -108,7 +108,6 @@ class AudioPlayerState: ObservableObject, Identifiable {
audioPlayerSubscription = nil audioPlayerSubscription = nil
audioPlayer = nil audioPlayer = nil
playbackState = .stopped playbackState = .stopped
showProgressIndicator = false
} }
func reportError(_ error: Error) { func reportError(_ error: Error) {
@ -143,13 +142,11 @@ class AudioPlayerState: ObservableObject, Identifiable {
} }
startPublishProgress() startPublishProgress()
playbackState = .playing playbackState = .playing
showProgressIndicator = true
case .didPausePlaying, .didStopPlaying, .didFinishPlaying: case .didPausePlaying, .didStopPlaying, .didFinishPlaying:
stopPublishProgress() stopPublishProgress()
playbackState = .stopped playbackState = .stopped
if case .didFinishPlaying = action { if case .didFinishPlaying = action {
progress = 0.0 progress = 0.0
showProgressIndicator = false
} }
case .didFailWithError: case .didFailWithError:
stopPublishProgress() stopPublishProgress()

View File

@ -285,7 +285,7 @@ class AudioPlayerStateTests: XCTestCase {
} }
audioPlayerActionsSubject.send(.didStartPlaying) audioPlayerActionsSubject.send(.didStartPlaying)
try await deferredPlayingState.fulfill() try await deferredPlayingState.fulfill()
XCTAssertTrue(audioPlayerState.showProgressIndicator) XCTAssertFalse(audioPlayerState.showProgressIndicator)
let deferred = deferFulfillment(audioPlayerState.$playbackState) { action in let deferred = deferFulfillment(audioPlayerState.$playbackState) { action in
switch action { switch action {
@ -300,6 +300,6 @@ class AudioPlayerStateTests: XCTestCase {
try await deferred.fulfill() try await deferred.fulfill()
XCTAssertEqual(audioPlayerState.playbackState, .error) XCTAssertEqual(audioPlayerState.playbackState, .error)
XCTAssertFalse(audioPlayerState.isPublishingProgress) XCTAssertFalse(audioPlayerState.isPublishingProgress)
XCTAssertTrue(audioPlayerState.showProgressIndicator) XCTAssertFalse(audioPlayerState.showProgressIndicator)
} }
} }

1
changelog.d/2190.feature Normal file
View File

@ -0,0 +1 @@
The voice message playback position indicator is displayed during playback or pause.