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

Support for AnnotationLists #4

Open
cubap opened this issue Feb 8, 2018 · 2 comments
Open

Support for AnnotationLists #4

cubap opened this issue Feb 8, 2018 · 2 comments

Comments

@cubap
Copy link
Contributor

cubap commented Feb 8, 2018

If you are interested, I announced Jack Reed's annotations from Slack on the Stanford Manifest (https://purl.stanford.edu/hg676jb4964/iiif/manifest) it was originally targeting. Currently, you only handle Ranges.

I may be bored later this week if you'd rather I pull request this.

@jeffreycwitt
Copy link
Owner

Supporting annotation list is definitely the next thing I want add. This way we can pull in transcriptions.

Let's do it.

I think there are some changes to be made to the plugin, since annotations apply to each canvas rather than the manifest, we either have to request all annotationsLists for the manifest at once and then only alert the user to their presence when they navigate to that canvas.

Or the plugin should be rechecking for notifications for each canvas.

@cubap
Copy link
Contributor Author

cubap commented Feb 8, 2018

I've never lifted the hood on Mirador myself, but I know smart people put it together, so I'll take an hour tomorrow and read up to see if I can spin this in for you.

The first, simple case, would use the same notification loader to get only those targeting the Manifest and load them into the otherContents instead of the structures. Best practice will need to be considered about transcription in general, since it may be more sensible to consider something like a Layer of annotations with a map of AnnotationLists keyed to canvas @ids. Mirador also probably has some considerations to make about some sort of viewing hint about on-screen display versus an index list in the side panel.

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

2 participants