emory-libraries / dlp-lux

Discovery for the DLP Cor Repository
2 stars 0 forks source link

Allow two-page layout view option in Universal Viewer #286

Closed nikdragovic closed 4 years ago

nikdragovic commented 4 years ago

User Story

As an end user, I want to see a side-by-side view of a text object such as a book in the Universal Viewer, so that I can view it in a context that approximates the physical object.

Recreating #276 due to apparent Zenhub issue.

Acceptance Criteria

Implementation note: A good source of documentation for UV configuration is https://github.com/UniversalViewer/universalviewer/blob/master/src/extensions/uv-seadragon-extension/config/en-GB.json. Example images below.

nikdragovic commented 4 years ago

Example object from Curate: https://curate-arch.library.emory.edu/concern/curate_generic_works/000000002j-cor?locale=en

nikdragovic commented 4 years ago

Screen Shot 2020-02-05 at 2.42.15 PM.png

Example of two up mode: https://bluemountain.princeton.edu/exist/apps/bluemountain/issue.html?titleURN=bmtnaae&issueURN=bmtnaae_1918-12_01#?#viewer&c=0&m=0&s=0&cv=0

nikdragovic commented 4 years ago

Page number data currently displaying in UV may help identify and properly display recto/verso.

Screen Shot 2020-02-06 at 5 24 58 AM
maxkadel commented 4 years ago

A working example of two up mode http://nlw.azurewebsites.net/?manifest=http%3A%2F%2Fwellcomelibrary.org%2Fiiif%2Fb19684915%2Fmanifest&locale=en-GB#?c=0&m=0&s=0&cv=1&xywh=-2%2C-1737%2C11931%2C10599

image

nikdragovic commented 4 years ago

Thanks @maxkadel , this looks like what we want.

maxkadel commented 4 years ago

With a properly formed manifest, the one or two page view choice shows in UV

Properly formed manifest requires

"@type":"sc:Sequence",
"viewingHint": "paged",

image

image

bess commented 4 years ago

This is done, IF the object from curate has a manifest that will support this.

Today (11 Feb) Max and Bess had a phone call with @devanshu-m about how to get the right settings added to the curate manifest. He said that the curate project is not making any changes to the content of iiif manifests until some decisions are made about how manifests will ultimately be generated. He is waiting on feedback from @rotated8 and Emily.

Should we call this done for now? Or should we put it on hold until we can get the curate manifests done?

eporter23 commented 4 years ago

@nikdragovic @maxkadel @rotated8 @devanshu-m @bess following up on a couple of conversations, if we now have UV support for this feature in Lux, and the dependency is on additional work from the Curate side, I think we can close this and revisit it later. We don't have a clear-cut way to manage some of this in Curate presently and want to focus on the higher priority manifest related work first.