search

hoarder-app / hoarder

A self-hostable bookmark-everything app (links, notes and images) with AI-based automatic tagging and full text search
https://hoarder.app
GNU Affero General Public License v3.0
6.48k stars 235 forks source link

Made the thumbnail image keeping its real ratio instead of object-cover like Pinterest layout #540

Open echo-saurav opened 1 month ago

echo-saurav commented 1 month ago
Screenshot 2024-10-15 at 9 32 53 PM
kamtschatka commented 1 month ago

one thing that recently happend was that those kinds of images get too big if you have full page screenshots active. In one of those cases the image height was 47000 px (or 7,6MB), so there definitely needs to be some kind of limit^^ See my PR, where I limited the max height to 1000px: https://github.com/hoarder-app/hoarder/pull/526

fyi: There is already an issue about this exact problem (https://github.com/hoarder-app/hoarder/issues/261), where we agreed that this needs fixing, so thanks for picking this up, but it needs some refinement.

echo-saurav commented 1 month ago

So i have change the max height to max-h-screen , i guess its enough ?