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

Bug: Set currentTime close to the end of video - video will end #952

Open
1 task done
maliuta-oleksandr opened this issue Jul 17, 2024 · 5 comments
Open
1 task done
Labels
bug Something isn't working Status: Unconfirmed A potential issue that we haven't yet confirmed as a bug

Comments

@maliuta-oleksandr
Copy link

maliuta-oleksandr commented Jul 17, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Which Mux Elements/Packages does this apply to? Select all that apply

mux-player-react

Which browsers are you using?

Chrome

Which operating systems are you using?

macOS

Description

This is simple video player. No styling, nothing - default progress bar

Try to click to 11 or 12 second - video will fire onEnd event.
If I want to change it using currentTime = value - result the same
But if you don’t change time - video plays well

Reduced test case

https://codesandbox.io/p/sandbox/mux-player-change-time-dp6k5p?file=%2Fsrc%2FPlayer.js

Steps to reproduce

  1. click to 11 or 12 second

Current Behavior

video will fire onEnd event

Expected Behavior

currentTime should be changed to new value

Errors

During video loading I see this (not when I change time)

image

What version of the package are you using?

2.8.0

@maliuta-oleksandr maliuta-oleksandr added bug Something isn't working Status: Unconfirmed A potential issue that we haven't yet confirmed as a bug labels Jul 17, 2024
@AdamJaggard
Copy link
Contributor

AdamJaggard commented Jul 18, 2024

Hey @maliuta-oleksandr, thank you for reporting this! I was able to see the issue you described on your code sandbox. We'll take a look at what might be the cause and add some additional details here if we find anything or get a fix working.

@maliuta-oleksandr
Copy link
Author

@AdamJaggard thanks!
And attaching code sample of how I upload videos, maybe there is something else, I came across that it could be issue with range headers
image

@maliuta-oleksandr
Copy link
Author

Hello, Is there any update?

@AdamJaggard
Copy link
Contributor

Hey @maliuta-oleksandr, we don't have a timeline for this to go out but it is being worked on currently. As soon as we have something to ship we'll post an update here to keep you updated.

@luwes
Copy link
Contributor

luwes commented Sep 18, 2024

I've run into this a few times myself. easy to reproduce with short videos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Status: Unconfirmed A potential issue that we haven't yet confirmed as a bug
Projects
None yet
Development

No branches or pull requests

3 participants