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

add story for non-interactive canvas #270

Merged
merged 3 commits into from
Aug 26, 2024

Conversation

abrin
Copy link
Contributor

@abrin abrin commented Jul 25, 2024

This PR is dependent on atlas-viewer/atlas#55, and aims to demonstrate that setting interactive=false in CanvasPanel disables the mousewheel and touch interactions

This PR is dependent on atlas-viewer/atlas#55, and aims to demonstrate that setting interactive=false in CanvasPanel disables the mousewheel and touch  interactions
Copy link

netlify bot commented Jul 25, 2024

Deploy Preview for iiif-canvas-panel failed.

Name Link
🔨 Latest commit 4237fa3
🔍 Latest deploy log https://app.netlify.com/sites/iiif-canvas-panel/deploys/66be73e46c3ed0000895fba8

Copy link

netlify bot commented Jul 25, 2024

Deploy Preview for canvas-panel-storybook failed.

Name Link
🔨 Latest commit 4237fa3
🔍 Latest deploy log https://app.netlify.com/sites/canvas-panel-storybook/deploys/66be73e4e69fcd00084033d0

Copy link

netlify bot commented Jul 25, 2024

Deploy Preview for iiif-canvas-panel-demos ready!

Name Link
🔨 Latest commit 4237fa3
🔍 Latest deploy log https://app.netlify.com/sites/iiif-canvas-panel-demos/deploys/66be73e48a7716000867eb40
😎 Deploy Preview https://deploy-preview-270--iiif-canvas-panel-demos.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

codesandbox-ci bot commented Jul 25, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 4237fa3:

Sandbox Source
simple-vue Configuration
vue-3-carousel Configuration
external-stylesheet-sandbox Configuration
example-sandbox Configuration
custom-preset Configuration
react-choices-example Configuration
reacting-to-the-user Configuration
external-annotation-pages Configuration
virtual-annotation-pages Configuration
loading-annotation-pages Configuration
annotation-display-1 Configuration
annotation-display-2 Configuration
choice-example Configuration
choice-react Configuration
more-regions-1 Configuration
more-regions-2 Configuration
regions-1 Configuration
regions-2 Configuration
regions-3 Configuration
regions-4 Configuration
regions-5 Configuration
regions-6 Configuration
responsive-1 Configuration

@abrin abrin marked this pull request as ready for review August 15, 2024 20:40
Comment on lines 29 to 31
"@atlas-viewer/atlas@https://pkg.csb.dev/atlas-viewer/atlas/commit/567c644a/@atlas-viewer/atlas":
version "2.2.6"
resolved "https://pkg.csb.dev/atlas-viewer/atlas/commit/567c644a/@atlas-viewer/atlas#767a57610e3e32ca8b1721644b56645732c7c034"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@stephenwf hmm... should this be 2.2.8?

Copy link
Member

Choose a reason for hiding this comment

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

Yes :/ yarn is a bit of a pain sometimes. It might work if you add it to the resolutions field.

I think for this repository I also need to publish a 2.0.15 that doesn't have the changes from #237

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think that worked, though it did update one of the helpers at the same time

@stephenwf stephenwf merged commit 794d4fe into digirati-co-uk:main Aug 26, 2024
5 of 13 checks passed
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.

2 participants