-
Notifications
You must be signed in to change notification settings - Fork 32
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
Added Theseus Viewer support #513
Conversation
We may need to remove the Label from the annotation as its confusing and doesn't add anything: https://preview.iiif.io/cookbook/feature/theseus-viewer/recipe/0135-annotating-point-in-canvas/ Currently viewer shows the label rather than the textual body but will show body once label has been removed. |
Look at this recipe: https://preview.iiif.io/cookbook/feature/theseus-viewer/recipe/0139-geolocate-canvas-fragment/ maybe remove the annotation label but go over the text. Viewer should show the property/@Label |
Add a note: https://preview.iiif.io/cookbook/feature/theseus-viewer/recipe/0230-navdate/ to say having navDate in the manifest isn't as useful as having it in the collection for viewers. |
Remove from first example: |
Add Newspaper |
Propose to add - 0154 Locate a Manifest on a Web Map |
Propose to add - 0240 Locate multiple canvases on a Web Map |
Propose to add 0074 Using Caption and Subtitle Files in Multiple Languages with Video Content |
Propose to add 0022 Linking with a hotspot |
Work to do to get this merged (to be confirmed in cookbook meeting): Cookbook group review:
Stephen:
|
just a note to remind me to tell stephen about UX inconsistency with 0154 (map in about pane) vs 0240 (map icon in left toolbar)... |
For Recipe 74 (#270), a person can change the caption language by changing the primary interface language in the "About this item" interface. However, the change only works for going from Italiano to English, not the reverse. Changing from Italiano to English also turns the CC off. Steps to reproduce:
The behavior is slightly different when changing the order of the language selected:
There's probably a bigger matrix to make to consider other/all possible paths, but perhaps these will suffice for now. |
Yes! The Manifest map is quite underwhelming, not much room to explore, especially if its a full GeoJSON geometry. Have thought about trying to make it so that you can "expand" and view it on the main area, but not found a nice way to do that. (I'm imagining that there might be cases where both the Manifest and Canvases have GeoJSON attached). @triplingual The main things I was aiming for:
But there is definitely work to do on the switching. Happy to go and make some improvements on that and mark this as partial support for now? |
I'm going to merge this in preperation for Trips recipe. |
https://theseusviewer.org/
Bringing #501 into main repo so we get the preview.