Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: 🐛 Enable seek gesture in paused mode #272

Merged

Conversation

himanshuGandhiSimform
Copy link
Collaborator

@himanshuGandhiSimform himanshuGandhiSimform commented Feb 28, 2024

This PR fix 200.

Copy link
Collaborator

@ujas-m-simformsolutions ujas-m-simformsolutions left a comment

Choose a reason for hiding this comment

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

Update change log, readme and format

hemantbeast added a commit to hemantbeast/audio_waveforms that referenced this pull request May 28, 2024
lib/src/audio_file_waveforms.dart Outdated Show resolved Hide resolved
lib/src/controllers/player_controller.dart Outdated Show resolved Hide resolved
@himanshuGandhiSimform himanshuGandhiSimform force-pushed the fix/200/enable_seek_gesture_in_paused_mode branch from cc18dea to 827191f Compare October 15, 2024 11:06
@@ -62,6 +62,9 @@ class AudioFileWaveforms extends StatefulWidget {
/// Allow seeking with gestures when turned on.
final bool enableSeekGesture;

/// Allow seeking with gestures on pause mode.
final bool enableSeekGestureOnPauseMode;
Copy link
Collaborator

Choose a reason for hiding this comment

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

we do not need this variable

Copy link
Collaborator Author

@himanshuGandhiSimform himanshuGandhiSimform Oct 15, 2024

Choose a reason for hiding this comment

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

so it means we didn't provide access to enable and disable this functionality? @ujas-m-simformsolutions

Comment on lines 267 to 270
if (progress < 0 ||
_playerState.isStopped ||
((_playerState.isPaused || _playerState.isInitialised) &&
!enableSeekGestureOnPauseMode)) return;
Copy link
Collaborator

Choose a reason for hiding this comment

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

This can be simplified as suggested before.

@himanshuGandhiSimform himanshuGandhiSimform force-pushed the fix/200/enable_seek_gesture_in_paused_mode branch 2 times, most recently from cf7c0c0 to fab7640 Compare October 15, 2024 11:56
@himanshuGandhiSimform himanshuGandhiSimform force-pushed the fix/200/enable_seek_gesture_in_paused_mode branch from fab7640 to 65b5674 Compare October 15, 2024 11:59
@ujas-m-simformsolutions ujas-m-simformsolutions merged commit 6995096 into main Oct 15, 2024
@ujas-m-simformsolutions ujas-m-simformsolutions deleted the fix/200/enable_seek_gesture_in_paused_mode branch October 15, 2024 12:00
hemantbeast pushed a commit to hemantbeast/audio_waveforms that referenced this pull request Jan 20, 2025
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.

enable seek gesture in paused mode
2 participants