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

Presentation v3 manifests should not require a separate thumbnail property on a canvas to display a thumbnail #593

Closed
1 of 3 tasks
jronallo opened this issue May 15, 2018 · 8 comments

Comments

@jronallo
Copy link

UV version:
Unknown. What version is http://universalviewer.io/examples running?

I'm submitting a:

Current behavior:

UV will not show a thumbnail in the left side panel for a v3 manifest unless the thumbnail property is populated for a canvas. Even if the canvas has an image annotation with an image service, UV will not show a thumbnail for the canvas.

Expected behavior:

This is something of a regression in behavior. UV should have some algorithm for determining an appropriate thumbnail to display. Here's a quick shot at what this might look like:

First look for a thumbnail property. If there is a thumbnail property then look for a service within. If there is an image service on the thumbnail use the service. If there is not an image service on the thumbnail use the id. If there is no thumbnail property then look for the first image annotation. If the resource has an image service use that. Otherwise fallback to the static image annotation.

Steps to reproduce:
This manifest works: http://universalviewer.io/examples/#?manifest=https%3A%2F%2Fd.lib.ncsu.edu%2Fcollections%2Fcatalog%2Fnubian-message-2003-04-01%2Fmanifest%3Fv3%3Dtrue

If you remove the thumbnail properties. it fails to show thumbnails in the side pane.

@tomcrane
Copy link
Contributor

@jronallo related: #102

@rosetta-development
Copy link

rosetta-development commented Jan 6, 2021

Hi,

Please take a look at -
IIIF-Commons/manifesto#24

rosetta-development added a commit to rosetta-development/universalviewer that referenced this issue Jan 6, 2021
fix(UniversalViewer#593): Presentation v3 manifests should not require a separate thumbnail property on a canvas to display a thumbnail UniversalViewer#593
@LlGC-szw
Copy link

All issues will be triaged for further investigation or closure by the 28 September 2023. If your issue is still relevant and would like for it be investigated further please comment by 14 September 2023.

@demiankatz
Copy link
Contributor

@edsilv, any idea if this is still an issue? I'll mark as ACTIVE until we can confirm.

@kirschbombe
Copy link

I think this issue was resolved with the work to support IIIF Image service?

@demiankatz
Copy link
Contributor

Does anyone have an example manifest without thumbnail properties that we can test with to confirm?

@kirschbombe
Copy link

kirschbombe commented Mar 13, 2024

Here is a cookbook recipe for a simple book manifest that has no thumbnail properties: https://iiif.io/api/cookbook/recipe/0009-book-1/. This generates thumbnails in UV 4.0.25: https://www.universalviewer.dev/#?xywh=-1565%2C-257%2C6333%2C5125&iiifManifestId=https%3A%2F%2Fiiif.io%2Fapi%2Fcookbook%2Frecipe%2F0009-book-1%2Fmanifest.json
I think it was fixed with #817

@demiankatz
Copy link
Contributor

Very helpful, @kirschbombe! Thank you! I'll close this as resolved.

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

No branches or pull requests

6 participants