Skip to content

Conversation

@Jason-Benson
Copy link
Contributor

Description of what you did:

@vercel
Copy link

vercel bot commented Nov 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
universalviewer Ready Ready Preview Comment Nov 10, 2025 5:53pm

@vercel
Copy link

vercel bot commented Nov 10, 2025

@Jason-Benson is attempting to deploy a commit to the mnemoscene Team on Vercel.

A member of the Team first needs to authorize it.

@demiankatz
Copy link
Contributor

Now here's an interesting puzzle. Take a look at this manifest: https://purl.stanford.edu/bh502xm3351/iiif3/manifest

It puts the same value in the metadata element for "description" and the overall "summary" element. As a result, fixing support for the Manifest v3 summary actually causes a duplicate "description" element to appear in the metadata sidebar.

Depending on how manifests are structured, this might or might not happen. A few thoughts:

1.) Maybe we should deduplicate the summary with metadata elements to explicitly prevent this specific scenario.

2.) Maybe we should rename "Description" to "Summary" so it's at least clear in this scenario that the duplicate data is serving two distinct purposes.

3.) Maybe we shouldn't display a summary at all, since the specification suggests that its purpose is to stand in for a longer description when space does not permit, and in the sidebar, we have enough space to show all the metadata -- but this could cause information loss if people interpret the spec differently, so option 1 seems preferable.

This also leads to the question of whether we should merge this fix as-is, on the understanding there may be undesirable side effects, or if we need to reach a consensus on this issue first!

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

Labels

None yet

Projects

Status: In testing

Development

Successfully merging this pull request may close these issues.

2 participants