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

markers plugin: markers at the begin/end of file were screwing up seeking #2208

Merged
merged 2 commits into from
Mar 8, 2021

Conversation

osheroff
Copy link
Contributor

@osheroff osheroff commented Mar 7, 2021

markers placed at the beginning or the end of a region weren't properly overflowing, instead they pushed the wavesurfer container to be wider and that messed up calculations when seeking around a waveform.

Does this need a CHANGELOG? not sure if you've released the marker stuff into the wild yet.

markers placed at the beginning or the end of a region weren't properly
overflowing, instead they pushed the wavesurfer container to be wider
and that messed up calculations when seeking around a waveform.
@coveralls
Copy link

coveralls commented Mar 7, 2021

Coverage Status

Coverage remained the same at 82.407% when pulling 2f81d59 on osheroff/seek_issue into 48e89e4 on master.

@thijstriemstra
Copy link
Contributor

thijstriemstra commented Mar 7, 2021

Does this need a CHANGELOG? not sure if you've released the marker stuff into the wild yet.

Always. And yea, it's included in 4.6.0.

@thijstriemstra thijstriemstra changed the title markers at the beginning/end of the file were screwing up seeking markers plugin: markers at the begin/end of file were screwing up seeking Mar 8, 2021
@osheroff
Copy link
Contributor Author

osheroff commented Mar 8, 2021

ok, changelog entry added.

Copy link
Contributor

@thijstriemstra thijstriemstra left a comment

Choose a reason for hiding this comment

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

Looks good. Squash merge away.

@osheroff osheroff merged commit d4ff489 into master Mar 8, 2021
sandiz pushed a commit to sandiz/wavesurfer.js that referenced this pull request Sep 1, 2021
…king (katspaugh#2208)

* markers at the beginning/end of the file were screwing up seeking

markers placed at the beginning or the end of a region weren't properly
overflowing, instead they pushed the wavesurfer container to be wider
and that messed up calculations when seeking around a waveform.

* add CHANGELOG entry
@katspaugh katspaugh deleted the osheroff/seek_issue branch May 28, 2023 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants