Skip to content

Presentation 4 #2316

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

Draft
wants to merge 85 commits into
base: main
Choose a base branch
from
Draft

Presentation 4 #2316

wants to merge 85 commits into from

Conversation

tomcrane
Copy link
Contributor

@tomcrane tomcrane commented Oct 24, 2024

The members of a Collection are typically listed in the `items` property or in a series of Collection Pages. The members _MAY_ include both other Collections and Manifests, in order, to form a tree-structured hierarchy. Collections without any members are allowed but discouraged. For example, a collection that had its last member removed might still be valuable to maintain as an empty collection.

If there are too many members in the collection to fit within a single document, then the members _MAY_ be listed in Collection Pages. A reference to the first page of members is given in the `first` property, and the last page in the `last` property. In this case, the Collection _MUST NOT_ use the `items` property. Collections with pages _MUST_ have at least two pages, otherwise the members should be included in `items` on the Collection. Collection Pages _MUST NOT_ be embedded within the Collection for the same reason.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MUST have at least two pages, otherwise the members should be included

Is this to avoid having first and last pointing to the same resource?

I wonder if this is a SHOULD; for dynamically-generated collections, you could be flip-flopping between items having two pages and a list of items. E.g.,

https://example.com/iiif/manifests-added-today

{: #position}

It is important to be able to position the (textual) body of an annotation within the Container's space that the annotation also targets. For example, a description of part of an image in a Canvas should be positioned such that it does not obscure the image region itself and labels to be displayed as part of a Scene should not be rendered such that the text is hidden by the three dimensional geometry of the model. If this property is not supplied, then the client should do its best to ensure the content is visible to the user.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs something to not give the impression that you have to "paint" renderings of textual bodies into the Container; if position not supplied the client will almost certainly render the body outside of a Canvas (e.g., in a side panel) and may even do similar for Scenes too.

{ "provides": [ "closedCaption" ] }
```

!!! warning "This breaks the graph as the file doesn't provide X in all contexts"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True, but... a machine-readable subtitles file, or a human-readable transcript file will very rarely be usable in any other context; it is a transcript of a particular thing, or is a set of subtitles for a particular thing, even when not linked to that thing; it still provides the function even when separated from the thing it is providing the function for.

(waves hands...)

Copy link
Contributor Author

@tomcrane tomcrane May 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's either this, or that same list become values for motivation (with some alterations)

"motivation": [ "providingSubtitles" ]

(sorry)

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

Successfully merging this pull request may close these issues.

6 participants