search

uidaholib / base-digital-collections-template

A highly customized version of CollectionBuilder-CSV used to generate University of Idaho Library Digital Collections.
MIT License
0 stars 0 forks source link

Cougar Dave browse blur on long PDFs #15

Open aweymo-ui opened 3 months ago

aweymo-ui commented 3 months ago

Running into image quality problems on the browse page of Cougar Dave for long PDF images. I replaced all smalls and thumbs with the updated rake task but there are still some blurry images on the browse page.

evanwill commented 3 months ago

@aweymo-ui the images look correct size. It maybe images cached in your browser? Did you test in a fresh private window on firefox?

aweymo-ui commented 2 months ago

Yep! Must have been a cache issue. I'll mark this as resolved.

evanwill commented 2 months ago

@aweymo-ui my only remaining question is if the really tall newspaper clipping images are disruptive to the flow of the Browse? I think there is just a couple of them, in which case: a) may not be worth doing anything, just leave them or b) could go manually edit the thumbnail images and crop them down to a reasonable size.

aweymo-ui commented 2 months ago

Creating unique derivatives feels like it might cause issues in the future. It seems like a good programmatic solution would be to add something to the browse layout that says, if a card is taller than x amount of pixels, automatically resize to x amount of pixels.

evanwill commented 2 months ago

you could limit the size displayed on Browse with CSS, but not a great solution because the user still has to load the full size image (which are bigger than normal), and then doesn't get to see it. Might add something to the Rake task in general, limiting the full height. But honestly, creating unique derivatives isn't a big deal--thats part of the flexibility of cb-csv, that you can pragmatically create custom stuff if it makes sense, rather than trying to account for every thing in your template.