Closed mathewjordan closed 2 years ago
Comments from cookbook editors:
In the first sentence: “You have a IIIF Manifest representing an object that is also represented by one or more web pages”
could you make it more explicit that this is the home page of the digital item. Could you also update the label in the manifest so that it mentions the home page of the digitial item as catalogue page may be more suitable in the seeAlso.
Could you change the example to use a image from https://fixtures.iiif.io rather than one hosted by Yale? Feel free to choose another one but this has a homepage at the Getty: https://fixtures.iiif.io/info.html?file=/images/Getty/laocoon.jpg.
Other things:
Cookbook editors agreed with the above changes it was OK to go through to the TRC.
Comments from cookbook editors:
In the first sentence: “You have a IIIF Manifest representing an object that is also represented by one or more web pages”
could you make it more explicit that this is the home page of the digital item. Could you also update the label in the manifest so that it mentions the home page of the digitial item as catalogue page may be more suitable in the seeAlso.
Could you change the example to use a image from https://fixtures.iiif.io rather than one hosted by Yale? Feel free to choose another one but this has a homepage at the Getty: https://fixtures.iiif.io/info.html?file=/images/Getty/laocoon.jpg.
Other things:
* Highlighting off in manifest. Should go to line 24. * No need for thumbnail in Manifest. * Add link to [metadata on any resource](https://preview.iiif.io/cookbook/0047-homepage/recipe/0029-metadata-anywhere/) in related recipes
Cookbook editors agreed with the above changes it was OK to go through to the TRC.
Thank you @glenrobson. The feedback has been applied and the Manifest now represents the Getty object you mentioned.
This PR addresses https://github.com/IIIF/cookbook-recipes/issues/47, setting the
homepage
property of a IIIF Manifest.