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

Support expandable images #31

Merged
merged 1 commit into from
Nov 25, 2024
Merged

Support expandable images #31

merged 1 commit into from
Nov 25, 2024

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Nov 20, 2024

Closes #26

Eject MDXComponents using Docusaurus swizzling. Adapt the expandable image logic from the Image component in gravitational/docs to the Docusaurus site, copying the ModalImage and PlainImage components, plus their supporting hooks, into the ejected MDXImg component.

@taraspos
Copy link
Contributor

taraspos commented Nov 21, 2024

@ptgott this seems to fail to deploy on amplify:

[cause]: Error: Expected component Admonition to be defined: you likely forgot to import, pass, or provide it.

Closes #26

Eject `MDXComponents/Img` using Docusaurus swizzling. Adapt the expandable
image logic from the `Image` component in `gravitational/docs` to the
Docusaurus site, copying the `ModalImage` and `PlainImage` components,
plus their supporting hooks, into the ejected `MDXImg` component.
@ptgott ptgott force-pushed the paul.gottschling/26-click-expand branch from 2aebb56 to 17eca09 Compare November 22, 2024 20:45
@gravitational gravitational deleted a comment from github-actions bot Nov 22, 2024
Copy link

github-actions bot commented Nov 22, 2024

🤖 Amplify preview here: https://paul-gottschling-26-click-expand.d2mrezcly5gcqm.amplifyapp.com

Preview details
  • LAST_UPDATED_AT: 2024-11-22T21:00:52.783000+00:00
  • JOB_ID: 0000000010
  • JOB_STATUS: SUCCEED
  • COMMIT_ID: 17eca09

@ptgott
Copy link
Contributor Author

ptgott commented Nov 22, 2024

@taraspos Thanks for catching the failure. The deployment seems to be working as expected now

@ptgott ptgott merged commit 450a8d7 into main Nov 25, 2024
4 checks passed
@ptgott ptgott deleted the paul.gottschling/26-click-expand branch November 25, 2024 13:10
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.

Images no longer "click-to-expand" leading to challenging readability
2 participants