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 array viewingHint #1557

Open
eroux opened this issue Nov 23, 2018 · 2 comments
Open

support for array viewingHint #1557

eroux opened this issue Nov 23, 2018 · 2 comments

Comments

@eroux
Copy link

eroux commented Nov 23, 2018

(see UniversalViewer/universalviewer#627), the following manifest fails:

http://eroux.fr/testmarcfail.json

because it has

"viewingHint":["continuous"]

but the spec says:

Any resource type may have one or more viewing hints.

so my understanding is that arrays are allowed as value for viewingHint.

@rsinghal
Copy link
Collaborator

@aeschylus We should confirm Manifesto handles this use case and then this issue will be resolved in Mirador 3.

@marlo-longley
Copy link
Member

I don't have a full answer to this, but I looked into it a bit ~

I do not see any examples of an Array being used for ViewingHint in Manifesto -- for example all the fixture files use single values.

Additionally, the way I am reading this, the vocabulary that Manifesto uses for its types only shows single values (e.g. "paged").

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

No branches or pull requests

4 participants