-
Notifications
You must be signed in to change notification settings - Fork 281
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
Segment Dragging will out of waveform. #547
Comments
Yes, it's inconsistent. You can drag a segment so that it ends after the right edge of the waveform view, but dragging the segment marker is limited to the width of the waveform view. |
The code in Peaks.js that handles this is a bit complex, and takes into account the pixel position of the markers, not just their time position. We could make the behaviour consistent (e.g, by allowing markers to be dragged beyond the visible window). But is this really a problem in practice? |
Thanks for reply. |
So, there are three separate changes we could make:
I'm hesitant about 2 and 3. |
Hello,
When enableSegmentDragging(true) is set, segment will not detect whether endTime is out of range.
1.mp4
The text was updated successfully, but these errors were encountered: