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

[Feature] GetThumbnail resolution too low #303

Closed
PenDrag0n7 opened this issue Jan 3, 2020 · 7 comments
Closed

[Feature] GetThumbnail resolution too low #303

PenDrag0n7 opened this issue Jan 3, 2020 · 7 comments
Labels
help wanted Extra attention is needed improvement Something needed tweaking.
Milestone

Comments

@PenDrag0n7
Copy link

PenDrag0n7 commented Jan 3, 2020

The resolution of gallerias thumbnails are a bit to low

100px shows up way too blur (365px would be nicer)
imaging.Resize(srcImage, 100, 0, imaging.NearestNeighbor)

@PenDrag0n7 PenDrag0n7 added the help wanted Extra attention is needed label Jan 3, 2020
@bnkai bnkai added the improvement Something needed tweaking. label Jan 3, 2020
@bnkai
Copy link
Collaborator

bnkai commented Jan 3, 2020

I wouldn't label that as a bug , maybe enhancement is better.
I thinks there is a tradeoff between how much,fast scrolling you need to do on the galleries page, and the image size , quality of the thumbnails.
I suppose we could change that to 200 or 300 and maybe also the resizer to lanczos or linear for a performance/quality aspect.
Anymore input is welcome though

@Leopere
Copy link
Collaborator

Leopere commented Jan 5, 2020

There are image optimization tools for this if we don't want to just simply restrict size.

@PenDrag0n7
Copy link
Author

maybe an override in the yaml could work

@Leopere
Copy link
Collaborator

Leopere commented Jan 6, 2020

Sounds a bit feature creepy but it would be good if it was variable.

@Orthodox-531
Copy link

Good thumbnails are a great thing to have for this use case (I use 400px in my main image manager). Since large images are expensive to scroll, perhaps it would make sense to favor pagination over large scrolling zones?

@Leopere
Copy link
Collaborator

Leopere commented Jan 28, 2020

Thumbnails will be super important I even see in a Version 2 feature-set where we use the metadata from the #334 O Counter helps determine screenshots from content to be used as the suggested thumbnail.

@Leopere Leopere changed the title [Bug Report] GetThumbnail resolution to low [Feature] GetThumbnail resolution to low Jan 28, 2020
@bnkai
Copy link
Collaborator

bnkai commented Feb 3, 2020

Closing as #336 is merged.

@bnkai bnkai closed this as completed Feb 3, 2020
@WithoutPants WithoutPants added this to the Version 0.1.0 milestone Feb 4, 2020
@Leopere Leopere changed the title [Feature] GetThumbnail resolution to low [Feature] GetThumbnail resolution too low Feb 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed improvement Something needed tweaking.
Projects
None yet
Development

No branches or pull requests

5 participants