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

IIIF Presentation 3 not supported #63

Open
keewerner opened this issue Feb 25, 2022 · 9 comments
Open

IIIF Presentation 3 not supported #63

keewerner opened this issue Feb 25, 2022 · 9 comments
Assignees

Comments

@keewerner
Copy link

All the examples at https://mirador-download-plugin.netlify.app/ use IIIF presentation level 2 examples. Manifests using IIIF level 3 instead don't work - the plugin loads but no actual page gets shown.

@massimozallocco
Copy link

I'm experiencing the same problem. Manifests IIIF level 3 not supported. The rendered link to images looks like "/full/640,/0/default.jpg?download=true" (missing image url).
Did anybody patched the error?
Thank you

@jeremy-liss
Copy link

I think this is the problem for us too. The download links point to a thumbnail URL, clicking on them opens a blank page. Has anyone managed to get it working with IIIF 3?

@keewerner
Copy link
Author

keewerner commented Mar 1, 2023 via email

@jeremy-liss
Copy link

Thanks for your help, I will investigate the share plugin.

@keewerner
Copy link
Author

keewerner commented Mar 1, 2023 via email

@jeremy-liss
Copy link

We have come across this plugin, it looks promising https://github.com/dbmdz/mirador-downloaddialog

@lutzhelm lutzhelm self-assigned this Nov 8, 2024
@lutzhelm
Copy link
Contributor

lutzhelm commented Nov 8, 2024

My guess is that this is a IIIF Image API 3 issue rather than a Presentation API problem. IIIF Image API does not support full size that is used in the code sample below, for example. I'll double check the image URI generation in all places, but it would be very welcome if any participants in this issue could provide any real world examples.

const boundsUrl = canvas
.getCanonicalImageUri()
.replace(
/\/full\/.*\/0\//,
`/${bounds.x},${bounds.y},${bounds.width},${bounds.height}/full/0/`,
);

@lutzhelm
Copy link
Contributor

There are some issues with manifesto's Canvas.getCanonicalImageUri method, e.g.

  • it does not provide canonical size parameters for IIIF Image API 3 in the first place (which are w,h instead of version 2's w,)
  • it relies on version 2 full size parameter that is not supported in version 3
  • it only provides a single image URI although there might be multiple images annotations or a Choice of images

Some of those issues might be fixable in manifesto, but a single image annotating the entire Canvas is only one (albeit extremely common) case, so it might be worth thinking about implementing the URI generation without manifesto.

@keewerner, @massimozallocco, @jeremy-liss: Sorry for the delay. I guess you all have found different solutions in the meantime, but could you still provide manifests where the plugin generated dysfunctional URIs? It would be very helpful to see if there are issues that I haven't identified yet.

@jeremy-liss
Copy link

@lutzhelm - it looks like this is working for our manifests now. I know that our manifests were updated at some point while we were working on rolling out Mirador, so I can't be certain whether it was updates to the plugin or manifest that fixed it

https://ndhadeliver.natlib.govt.nz/delivery/iiif/presentation/3/IE18987210/manifest

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

4 participants