-
Notifications
You must be signed in to change notification settings - Fork 17
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
Tryed your example site #1
Comments
This makes me think my work with |
I am using windows 10, Internet Explorer 11. |
I just emailed Browserstack asking for a free open source account. I'll be able to test this myself as soon as I hear back. In the meantime I did some googling and it does look like IE has some issues with localStorage. I'll likely need to end up using a wrapper like https://github.com/marcuswestin/store.js. Check back soon! |
Thanks. |
I just pushed a new version. Take a test. (see updated README) |
Yes it nows works. A few bugs. The bugs are sometimes it wont show the resume button even though your a bit in the video. When it does resume it may miss a few seconds even before or after. |
The example sets |
I removed |
Thanks. When you refresh the browser twice the resume button shows when you press it goes back to start if you refresh once and you press the resume button it works if you refresh more then once the resume button shows but doesn't work as expected. |
I'm testing in IE11 on Win 10 using the current demo link from the Readme. I seek ahead to the middle of the video while it's playing. Then I refresh it multiple times. No matter how many times I refresh it it always picks up playing where I expect it to. Not sure how else I can test this. |
@paladox I'd really like to make sure this is working for all modern browsers. Is there any chance you could record a screen cast of it not working so I could get a better idea of the problem? |
Ive tried you example site but when I refresh it still goes to the start.
The text was updated successfully, but these errors were encountered: