Skip to content

Conversation

@veroandreo
Copy link
Contributor

The .lightbox class allows you to click on images to see a larger version of them. It's much needed, especially for GUI screenshots, when users should be able to see and read what's there 🤓

image

@veroandreo veroandreo requested review from cwhite911 and landam May 21, 2025 19:31
@veroandreo veroandreo self-assigned this May 21, 2025
Copy link
Contributor

@cwhite911 cwhite911 left a comment

Choose a reason for hiding this comment

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

If you want to enable lightbox for all images in the document you can set lightbox: true in the header, so you don't need to add the .lightbox class to each image individually.

---
lightbox: true
---

Also, order of attributes in divs and spans matter to Pandoc. I commented on one instance but in most cases you should move the .lighbox class before any of the attributes like width or fig-align.

From the docs
[This is good]{#id .class key1="val1" key2="val2"}
[This does not work!]{.class key="val" #id}

@veroandreo
Copy link
Contributor Author

If you want to enable lightbox for all images in the document you can set lightbox: true in the header, so you don't need to add the .lightbox class to each image individually.

---
lightbox: true
---

Also, order of attributes in divs and spans matter to Pandoc. I commented on one instance but in most cases you should move the .lighbox class before any of the attributes like width or fig-align.

From the docs
[This is good]{#id .class key1="val1" key2="val2"}
[This does not work!]{.class key="val" #id}

Ah! When I went through those instructions, I did not understand them. Thanks for explaining! Will update accordingly.

@veroandreo veroandreo merged commit 37a4448 into OSGeo:main May 23, 2025
1 check failed
@veroandreo veroandreo deleted the clickable-imgs branch May 23, 2025 19:14
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