Skip to content
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

compound object viewer #68

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Natkeeran
Copy link
Contributor

@Natkeeran Natkeeran commented Mar 2, 2023

Please see the following video that explains the proposed approach for a compound viewer: https://drive.google.com/drive/u/0/folders/1oJmAsf8KHt-1YhJ3NSPS_bJ5eTRrNJ2_

Compound Object Viewer provides a lightweight approach to display media associated with compound object’s members, and to navigate between them.

It uses two views, one to set up the service file display, and other to setup the Compound Object Members thumbnail navigation.

A hook function is used to load the media or the service file of the first member object. The user can select to view any other member media by clicking on the thumbnail navigation below. Currently, the members are sorted by node id; however, other fields such as the field_wieght can be used to sort the order of the members as well.

A javascript function is used to highlight the selected member object.

Differences with the current sandbox compound viewer version

  • When the user clicks on the compound object, it does not go to the first member object node. It goes to the compound object node, then loads the first member's service file.
  • Currently, it does not provide Item and Set descriptions. However, this can be added on after a metadata display approach is selected for the islandora starter site.

Known Issues

  • Currently, if the member media does not have thumbnail, it does not load get loaded in the navigation block. This will be addressed.
  • Accessibility of the navigation panel needs to be tested, and aria labels would need to be added.

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.

1 participant