Skip to content

Conversation

@t11r
Copy link
Contributor

@t11r t11r commented Oct 7, 2025

This adds TIFY to each supported recipe of the IIIF Cookbook.

Website: https://tify.rocks/
Code repository: https://github.com/tify-iiif-viewer/tify

@glenrobson
Copy link
Member

glenrobson commented Oct 10, 2025

Thank you @t11r for submitting this pull request and we look forward to adding TIFY to the list of viewers. There were a couple of recipes that we didn't think TIFY supported:

  • Image disapears in this example: 🔗 0014-accompanyingcanvas but it should be shown while the audio is playing.
  • 0035-foldouts the second "foldout folded" canvas should be shown alongside the title page not as an individual

We thought TIFY supported this recipe but it wasn't listed:

And we took the following notes which won't stop us merging the pull request:

  • 🔗 0234-provider doesn't include a link to the Provider seeAlso but OK to merge.
  • 🔗 0283-missing-image we see error dialog but shows the last image that was looked at. Would be nicer if it showed a broken image like it does for the thumbnails.

@triplingual triplingual added the viewer-change Add/remove/change viewer entirely or for recipe support label Oct 13, 2025
@t11r
Copy link
Contributor Author

t11r commented Oct 17, 2025

Thank you for reviewing, @glenrobson! I added TIFY to the second example of 0010 and removed it from 0014 and 0035. I will add those with a separate PR once the issues are fixed.

@glenrobson glenrobson added the meta: ready-to-merge Pull request is ready to merge into main branch label Oct 24, 2025
@glenrobson
Copy link
Member

Thanks for the changes. Merging.

@glenrobson glenrobson merged commit 7addb9d into master Oct 24, 2025
1 of 2 checks passed
@glenrobson glenrobson deleted the introduce-tify branch October 24, 2025 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta: ready-to-merge Pull request is ready to merge into main branch viewer-change Add/remove/change viewer entirely or for recipe support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants