-
Notifications
You must be signed in to change notification settings - Fork 844
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
[photos] Best practice to use lightbox #1056
Comments
A great question, to which I'd also like to know the answer. The documentation makes it seem like using Edit: I just discovered https://github.com/kura/lightbox/, and added it to my |
I agree, it's far from a good out-of-the-box experience. There are some instructions in the Photo plugin README.md about modifying the Jinja templates, which is where I started by following their Magnific popup examples. But after that experience I ended up do a fairly significant overhaul to both the plugin and the templates to replace Magnific with PhotoSwipe. |
Update: I didn't use the Kura lightbox in the end, it required too much heavy lifting. I eventually understood how to integrate |
@antgel: Based on what you learned, would you be willing to make some edits/additions to the Photos plugin README to help clarify and elaborate? |
Glad you got it working. The README probably should be more explicit that you need to include the JS and CSS for http://lokeshdhakar.com/projects/lightbox2/ yourself, as using |
We have moved the plugin to its own repository(https://github.com/pelican-plugins/photos/) and I have created an issue(pelican-plugins/photos#1) there to track the process. Feel free to use the new issue to add comments. Contributions are welcome. @justinmayer or @sprnza I think you can close this issue. |
Agreed — let's move this discussion to the aforementioned issue in the new independent repository. |
What is the best practice to use lightbox using this syntax
![]({lightbox}folder/file)
?The issue is in the README there is described a way to add Magnific Popup css and js files for templates that have a gallery or an image attached only and it obviously doesn't work for images added in the body of my text. Should I add the style and the javascript in every article or there is a way to find the
{lightbox}
syntax used on the page?The text was updated successfully, but these errors were encountered: