Skip to content

feat: add click-to-zoom lightbox for doc images#97

Merged
zoltan-baba merged 2 commits into
mainfrom
feat/image-lightbox
Jul 9, 2026
Merged

feat: add click-to-zoom lightbox for doc images#97
zoltan-baba merged 2 commits into
mainfrom
feat/image-lightbox

Conversation

@zoltan-baba

Copy link
Copy Markdown
Collaborator

Summary

  • Wire in docusaurus-plugin-image-zoom so readers can click any doc image to view it enlarged, instead of being stuck at inline size.
  • Scope the zoom selector to skip images that are already wrapped in a link (e.g. the MCP install badge), so those keep navigating normally instead of opening the lightbox.
  • Reuse the site's --ifm-background-color var for the zoom overlay background instead of a hardcoded color, so it can't drift from the theme.

Test plan

  • Verified locally: clicking a doc image opens the zoomed overlay, background matches the site's surface color, and click-away closes it.
  • Confirmed linked images (e.g. the MCP install badge) still navigate on click instead of zooming.
  • npm run build completes successfully.

zoltan-baba and others added 2 commits July 9, 2026 12:02
Wire in docusaurus-plugin-image-zoom so readers can click any doc
image to view it enlarged, since screenshots were previously only
viewable at their inline size.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The zoom selector matched every img in .markdown, including the one
markdown image that's wrapped in a link (the MCP install badge), so
clicking it opened the lightbox instead of navigating cleanly. Also
swap the hardcoded dark background for the existing --ifm-background-color
var so it can't drift from the site's actual theme colors.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

@zoltan-baba zoltan-baba merged commit 1904c14 into main Jul 9, 2026
1 check passed
@zoltan-baba zoltan-baba deleted the feat/image-lightbox branch July 9, 2026 13:19
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