0540 link for opening multiple canvases #564
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ultiple-canvases 0540 link for opening multiple canvases
…ultiple-canvases Fix typos and add link
…ultiple-canvases remove inline JSON
…ultiple-canvases changes id.url to id.path
…ultiple-canvases fix broken link
markpbaggett
reviewed
Dec 3, 2024
| "type": "Canvas", | ||
| "partOf": [ | ||
| { | ||
| "id": "{{ id.path }}/manifest", |
There was a problem hiding this comment.
This results in a 404 and this property in the json:
"id": "https://preview.iiif.io/cookbook/0540-link-for-opening-multiple-canvases/recipe/0540-link-for-opening-multiple-canvases/canvas/p2",
I think we need this:
Suggested change
| "id": "{{ id.path }}/manifest", | |
| "id": "{{ id.path }}/manifest.json", |
|
|
||
| ## Implementation Notes | ||
|
|
||
| Some viewers already implement custom formats for exporting the current workspace for sharing or later use. Content State API could be used for the same purpose, adding the advantage of direct loading of the workspace using a crafted link with the `iiif-content` query parameter. The [multiple targets for a comparison view section](https://iiif.io/api/content-state/1.0/#53-multiple-targets-for-a-comparison-view) describes a method for targetting two Canvases at the same time; each Canvas could be from a different Manifest. |
There was a problem hiding this comment.
Minor suggestion:
Suggested change
| Some viewers already implement custom formats for exporting the current workspace for sharing or later use. Content State API could be used for the same purpose, adding the advantage of direct loading of the workspace using a crafted link with the `iiif-content` query parameter. The [multiple targets for a comparison view section](https://iiif.io/api/content-state/1.0/#53-multiple-targets-for-a-comparison-view) describes a method for targetting two Canvases at the same time; each Canvas could be from a different Manifest. | |
| Some viewers already implement custom formats for exporting the current workspace for sharing or later use. The Content State API could be used for the same purpose, adding the advantage of direct loading of the workspace using a crafted link with the `iiif-content` query parameter. The [multiple targets for a comparison view section](https://iiif.io/api/content-state/1.0/#53-multiple-targets-for-a-comparison-view) describes a method for targetting two Canvases at the same time; each Canvas could be from a different Manifest. |
| Some viewers already implement custom formats for exporting the current workspace for sharing or later use. Content State API could be used for the same purpose, adding the advantage of direct loading of the workspace using a crafted link with the `iiif-content` query parameter. The [multiple targets for a comparison view section](https://iiif.io/api/content-state/1.0/#53-multiple-targets-for-a-comparison-view) describes a method for targetting two Canvases at the same time; each Canvas could be from a different Manifest. | ||
|
|
||
| For this purpose, we create an Annotation with `motivation` set to `["contentState"]`. | ||
| The value of the the `target` attribute of the Annotation is a list containing the `id` of the Canvases and a `partOf` attribute with the `id` of the Manifests they belong to. |
There was a problem hiding this comment.
Would property be better than attribute here?
Suggested change
| The value of the the `target` attribute of the Annotation is a list containing the `id` of the Canvases and a `partOf` attribute with the `id` of the Manifests they belong to. | |
| The value of the the `target` property of the Annotation is a list containing the `id` of the Canvases and a `partOf` property with the `id` of the Manifests they belong to. |
|
The cookbook editors have reviewed this receipe and are happy for it to go through with the following changes:
Example
Related recipe
|
Co-authored-by: Mark Baggett <[email protected]>
…ultiple-canvases 0540 link for opening multiple canvases
…ultiple-canvases change related recipes and corrects typos
…ultiple-canvases change url formatting and correct typo
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Recipe link: https://preview.iiif.io/cookbook/0540-link-for-opening-multiple-canvases/recipe/0540-link-for-opening-multiple-canvases/