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

Treat frame durations <= 10 ms as if they were 100 ms #108

Merged
merged 1 commit into from
Jan 8, 2023

Conversation

qbnu
Copy link
Contributor

@qbnu qbnu commented Jan 8, 2023

The behavior of JPEGView differs from the major web browsers when displaying animated images with frame durations <= 10.
Currently, JPEGView displays all 0ms GIF frames as 100ms, and all <10ms frames (for other formats) as 10ms.
Most web browsers display all <=10ms frames as 100ms:

Some examples that display differently in JPEGView:
0-delay APNG
orbit
1-delay GIF (centiseconds)
dancing-dog-1

Some relevant posts:
The Fastest GIF Does Not Exist
Animated GIF Minimum Frame Delay Browser Compatibility Study

@sylikc
Copy link
Owner

sylikc commented Jan 8, 2023

Thanks for including links and info. I'm learning something new about these animated formats that I haven't ever even thought about, even though things like GIFs are so ubiquitous. It's funny to watch the dog go wild in the current builds 👀

@sylikc sylikc merged commit 6c83601 into sylikc:master Jan 8, 2023
@sylikc sylikc added the format support Related to add/remove/change of a specific format support. label Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
format support Related to add/remove/change of a specific format support.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants