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

Prevent fractions of thumbnails. #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

gregdick1
Copy link

The fractional thumbnail on the right end of the bar was really bugging me, so I set out to try and prevent fractional thumbnails by calculating the closest width necessary to perfectly fit the thumbnails into the bar. Some thumbnails will end up with a picWidth +1 larger than others to make up for fractions. This works regardless of what picWidth you set.

Old bar with fractional thumbnail on the end (screen taken from readme.md)
savideorangeslider0

New bar with no fractional thumbnails
savideorangeslider1
savideorangeslider2
savideorangeslider3

(note the difference in bar lengths is just due to how the screenshots were taken. It's not from any changes in the code)

@dimohamdy
Copy link

@andrei200287 where are you please check pull requests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants