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

feat: image viewer controls (zoom & fullscreen) #2451

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

Conversation

lbiedinger
Copy link
Member

@lbiedinger lbiedinger commented Oct 10, 2024

Adds a MediaImageViewerControls component to be used in the ItemMediaPresentation.

  • This way it can be added to the iiif-viewer-toolbar where it is conditionally toggled for image type resources.
  • Data attributes to manage zoom related variables are added to ItemMediaPresentation.
  • Events for updates to the zoom levels of the openLayers map are added to the MediaImageViewer, this way the zoom levels remain synchronized.

Fullscreen support is handled in ItemMediaPresentation as this contains the ref to the element we want to make fullscreen.

  • In fullscreen mode the thumbnail buttons & bar are hidden via css rules, to prevent users from switching to a non-image resource which would not contain a fullscreen toggle as this is part of the MediaImageViewerControls.

Copy link

Preview deployment: https://pr-2451.portal-js.dev.eanadev.org/

Copy link

sonarcloud bot commented Oct 11, 2024

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