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

fix: Thumbails have correct aspect ratio and reload if loaded while b… #630

Merged
merged 12 commits into from
Jan 31, 2025

Conversation

woelper
Copy link
Owner

@woelper woelper commented Jan 29, 2025

…eing written

@Stoppedpuma
Copy link
Collaborator

The "Open thumb dir" button seems to freeze Oculante until the opened file manager window is closed.

@Stoppedpuma
Copy link
Collaborator

@woelper Another thing which you should probably check out before merging is #633.

@Stoppedpuma
Copy link
Collaborator

Stoppedpuma commented Jan 30, 2025

#633 Still happens around the actual image part:

image

The rounded corners on thumbnails also seem to have regressed on this PR. This seems to be caused by 61cbce7. The correct rounding value should be 6.

@woelper
Copy link
Owner Author

woelper commented Jan 30, 2025

I am using the exact same rect, maybe this is because the image is snapped / aligned to whole pixels?

@woelper
Copy link
Owner Author

woelper commented Jan 30, 2025

It seems rounding is not an option due to how the image is drawn. I can look at it, but it will likely not be possible.

@woelper
Copy link
Owner Author

woelper commented Jan 30, 2025

#633 Still happens around the actual image part:

image

The rounded corners on thumbnails also seem to have regressed on this PR. This seems to be caused by 61cbce7. The correct rounding value should be 6.

Fortunately I was wrong. To me it looks good now, can you double check please?

@Stoppedpuma
Copy link
Collaborator

This looks good to me as well! Thanks for taking the time to look into the rounding as well as the issue at #633!

One thing which sticks out to me that isn't really too much of an issue is that the new hover brightness value makes basically every images look either too bright or kind of washed out, Was there a reason for this change outside of the thumbnails?

@woelper
Copy link
Owner Author

woelper commented Jan 31, 2025

This looks good to me as well! Thanks for taking the time to look into the rounding as well as the issue at #633!

One thing which sticks out to me that isn't really too much of an issue is that the new hover brightness value makes basically every images look either too bright or kind of washed out, Was there a reason for this change outside of the thumbnails?

There wasn't a reason besides me thinking that the effect was too subtle. I reverted the change to 5.

@woelper woelper merged commit a709743 into master Jan 31, 2025
8 checks passed
@woelper woelper deleted the fix_thumbs branch January 31, 2025 23:53
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