Document how to group photoswipe images into a gallery in views using photoswipe-gallery wrapper class

Created on 13 November 2023, 8 months ago

Missing Next and Prev arrows when Photoswipe using in view fields.
I try both versions 4 and 5.

πŸ’¬ Support request
Status

Needs work

Version

5.0

Component

Documentation

Created by

πŸ‡ΊπŸ‡¦Ukraine VasyOK

Live updates comments and jobs are added and updated live.
  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Comments & Activities

  • Issue created by @VasyOK
  • Assigned to thomas.frobieter
  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica

    Thanks for raising this. It's not a bug, but there's missing information in the README.md that needs to be added to show images from a Drupal view in a Photoswipe Gallery.

    If not adding a photoswipe-gallery wrapper class in the view around, the photoswipe images won't be shown as gallery, but as single pictures. That's expected, but we should document how to group them into a gallery by adding the required photoswipe-gallery class.

    @thomas.frobieter could you elaborate on that in a MR for the README.md? There's already a views subheading, which I think we should extend with the info where the class can / should be put.

    I think it would be in the image field or in the views style?

    Don't think we'll ever be able to add a UI for that while keeping the required flexibility.

  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica

    TL;DR: Add the class photoswipe-gallery on the view or another dom element around the images you want to group into a gallery.

  • thomas.frobieter β†’ made their first commit to this issue’s fork.

  • @thomasfrobieter opened merge request.
  • Status changed to Needs review 8 months ago
  • Status changed to Needs work 8 months ago
  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica

    @thomas.frobieter nice! If possible could you please:
    - Use code style for the class snippets
    - Eventually add a code example if someone advanced wants to do it code-wise in a template?
    - But more important, I think: Additionally to the general explanation, perhaps add the steps a user has to take in the Views UI to add these classes (for less advanced users)

    Perhaps @LRWebks can help you with details?
    This has low priority, but I think we should help newbies as good as possible.

    Perhaps we should generally review the README and check if it explains the general concepts good enough for users starting with PS?

    BTW. when using a single image field with the photoswipe formatter, I think these will be grouped, but not for cases where you'd like to have a gallery across fields or view results?

  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica

    PS: Of course, community help with the documentation is highly appreciated.

  • Issue was unassigned.
  • Well, I think it's good enough. I won't invest more time into this topic.

  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica

    Tagging for community help to add the steps to be taken in the View UI to add the classes there. @thomas.frobieter added the required general steps / themer steps.

  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica
  • πŸ‡ΊπŸ‡¦Ukraine VasyOK

    Thanx, @Anybody, you right!

    I add class "photoswipe-gallery" to view field - and galleries worked!

Production build 0.69.0 2024