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

Include rights and requiredStatement in metadata display component #331

Closed
3 tasks done
elynema opened this issue Jan 8, 2024 · 8 comments
Closed
3 tasks done

Include rights and requiredStatement in metadata display component #331

elynema opened this issue Jan 8, 2024 · 8 comments
Assignees

Comments

@elynema
Copy link

elynema commented Jan 8, 2024

Description

This suggestion comes from the January IIIF A/V community call. For those who use rights and requiredStatement, they would probably like to have those displayed in the metadata component.

requiredStatement MUST be displayed, so that's a IIIF requirement.

rights is intended to be a machine-actionable uri, so we probably don't need to include that in metadata display: https://iiif.io/api/cookbook/recipe/0008-rights/

Done Looks Like

  • Add a heading at the end of the Metadata that says "Rights" like how Mirador does it. If they exist, requiredStatement and rights could show up below that.
  • requiredStatement added to descriptive metadata display component. The labeled in the requiredStatement displays as the label for the text in the Metadata display.
  • rights added as a URI to the metadata display component (with label of "License")
@elynema
Copy link
Author

elynema commented Feb 23, 2024

@Dananji Dananji self-assigned this Apr 15, 2024
@elynema
Copy link
Author

elynema commented Apr 17, 2024

Dananji has styled this to be similar to Mirador.

@Dananji
Copy link
Collaborator

Dananji commented Apr 26, 2024

This can be tested on Ramp demo site.

@elynema
Copy link
Author

elynema commented Jun 27, 2024

@Dananji Do you have an example of an accessible manifest you were using for test that has these components?

@Dananji
Copy link
Collaborator

Dananji commented Jul 3, 2024

I used the demo manifest in Ramp which only has the changes in local dev environment.
Maybe we could use the Aviary manifests in the above comment?

@Dananji
Copy link
Collaborator

Dananji commented Jul 3, 2024

@elynema
Copy link
Author

elynema commented Jul 3, 2024

This is working fine for the examples Dananji listed.

@Dananji is the code pulling in the label in the requiredStatement as the display label? All the examples listed here use the label "Attribution" so it's hard to tell.

@elynema elynema closed this as completed Jul 3, 2024
@Dananji
Copy link
Collaborator

Dananji commented Jul 3, 2024

Yes the labels in the requiredStatement are used in the display.

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