-
Notifications
You must be signed in to change notification settings - Fork 16
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
Option to customise thumbnail display #223
Comments
Hi @lukew-cogapp, thank you for submitting! I like adding both options, perhaps extending the {
options: {
thumbnails: {
style: "auto" | "hide" | "icons"; // default: "auto"
}
}
} |
Some generic default with an option to pass a URL perhaps, for something custom. Might ask our designer to have a quick think! |
We'll send over a mockup for this :) I also note that if a manifest does not specify thumbnails, the viewer attempts to load the info.json, presumably, it tries to use the FYI, because I checked:
I can see two options:
I think creating thumbnail URLs on the fly is the best route, but ideally in combination with the Edit: I think it's the logic here that would need modifying, maybe just something as simple as checking for clover-iiif/src/hooks/use-iiif/getThumbnail.ts Lines 41 to 45 in 43da0ee
|
A 👍 to dynamically generating thumbnails. iiii-helpers / Vault has a helper function for this: https://github.com/IIIF-Commons/iiif-helpers/blob/main/docs/thumbnails.md?plain=1 |
Hello, enhancement ideas for handling thumbnails:
The text was updated successfully, but these errors were encountered: