Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

evaluate IIIF image options #28

Open
evanwill opened this issue Jan 21, 2020 · 2 comments
Open

evaluate IIIF image options #28

evanwill opened this issue Jan 21, 2020 · 2 comments

Comments

@evanwill
Copy link
Contributor

CONTENTdm implementation of IIIF has some limitations.

currently:

  • image includes use pct: with % set in the theme.yml, e.g. large pct:70. this works consistently with most collections, unless the images in CDM are abnormally large or small. However, using % is a bit random since you don't know the starting size. Secondly, CDM does not seem to allow less than 10%.
  • carousel include uses just the height, ,h based on the height of the carousel. This ensures the image makes sense in that location, but can end up with odd sizes for images with extreme ratios, large landscape or large vertical images.

going forward:

  • best option would be !w,h which should give something scaled to best fit inside those dimensions, but CDM is just returning max instead. so unfortunately this doesn't work.
  • do we want to switch from pct: to height or width?
@evanwill
Copy link
Contributor Author

evanwill commented Jan 21, 2020

@owikle @dcnb we should think about this, and then document how the API works and what to consider if it isn't working for a specific collection.

I would also like to figure out a way to create a recipe for the API call for use in JS includes. And maybe just restructuring the image includes in general to make it easier to swap out APIs.

I added a bit more info to the docs:

https://github.com/CollectionBuilder/collectionbuilder-contentdm/blob/master/docs/contentdm-api.md

@evanwill
Copy link
Contributor Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant