Skip to content

Recipe #560: Resources on a Timeline #562

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

Open
wants to merge 30 commits into
base: master
Choose a base branch
from

Conversation

triplingual
Copy link
Contributor

Closes #560

Note that there's an arbitrary time-based resource in the Manifest. This is to try to force a viewer to create a timeline that the still image resources can show up along. (In other words, a hack for the time being)
@triplingual triplingual marked this pull request as draft November 22, 2024 15:36
The repeat behavior I think I want is only legitimate on a Manifest or Collection, which necessitates multiple Canvases
Also cleaned up some IDs, but that's not the headline
@glenrobson
Copy link
Member

glenrobson commented Mar 17, 2025

The cookbook editors are happy for this to go through but asked if you could look at the following:

Implementation notes

  • “the viewer is expected to provide a means of displaying that Canvas...” could you be less directive as the cookbook generally encoruages viewers rather than directs what they should do.
  • “Nothing in the spec says anything explicitly about a viewer” - remove link to spec as it doesn't link to a specific section.
  • There was discussion whether the t parameter should be in quotes rather than backticked
  • “it can use a time fragment on its target value, in the form #t=” could you use media fragment instead of time fragment as this is the term in the mediafragments specification.
  • “This parameter’s value can be a range, a starting time (to indicate the resource should display until the end of the duration), or an ending time (to indicate the resource should display from the beginning of the duration).” - For details see the media fragment spec when specifying one number. If we have to have it include the table from the media frags from spec.
  • Add to the “Nothing in the spec paragraph” that other values are allowed in the t parameter but conventionally seconds are used. See slack discussion: https://iiif.slack.com/archives/C18FHBKAQ/p1663762334826879 and IIIF api issue
  • Suggested change: “To get the Canvas to repeat the sequential items display, the behavior value used is repeat” “To get the annotation sequence to repeat the behavior value is set to “repeat”.
  • Suggested change: “Within a Canvas’s items the parameterized target values tell viewers” - Within a Canvas’s annotations the parameterized target values tell viewers

Example

The images in the manifest has different aspect ratios so you should include the note from the linked recipe: “The aspect ratio should be consistent between your source image and Canvas. Otherwise, you’ll see unpredictable stretching and/or distorting.”

@glenrobson glenrobson added the meta: ready-for-trc Issue is ready for TRC review label Mar 18, 2025
@triplingual
Copy link
Contributor Author

Just to note that I will start tackling this 20 March.

@triplingual
Copy link
Contributor Author

triplingual commented Mar 20, 2025

  • less directive
  • remove link to spec
  • t parameter in quotes rather than backticked
  • "media fragment" instead of "time fragment"
  • "For details see the media fragment spec when specifying one [value]"
  • Add that conventionally seconds are used
  • “To get the annotation sequence to repeat the behavior value is set to 'repeat'."
  • "Within a Canvas’s annotations the parameterized target values tell viewers . . ."

@triplingual
Copy link
Contributor Author

Changes made per IIIF/trc/issues/132 comments

@glenrobson glenrobson added meta: ready-to-merge Pull request is ready to merge into main branch and removed meta: ready-for-trc Issue is ready for TRC review meta: ready-for-editors Ready for Cookbook editor review labels Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta: ready-to-merge Pull request is ready to merge into main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Displaying resources along a timeline independent of time-based resources
2 participants