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

Added custom PID support per image. Added line to documentation accordingly. #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

stamminator
Copy link

Now the data-pid attribute may be specified on the img tag. Added line in readme.md about the data-pid attribute.

ISSUES:

  1. Currently works whether or not the galleryPIDs: true option is set.
    Not sure if that's correct or not.
  2. If galleryPIDs option IS set to true, then the page should
    automatically open the image with the custom PID in the URI, but this is
    not working properly yet. Only default PIDs will open. If a custom PID
    is in the URI when the page loads, as soon as the user clicks on ANY
    gallery image, the image with that custom PID will open, rather than the
    one the user clicked on.

Now the `data-pid` attribute may be specified on the `img` tag.

ISSUES:
1. Currently works whether or not the `galleryPIDs: true` option is set.
Not sure if that's correct or not.
2. If `galleryPIDs` option IS set to `true`, then the page should
automatically open the image with the custom PID in the URI, but this is
not working properly yet. Only default PIDs will open. If a custom PID
is in the URI when the page loads, as soon as the user clicks on ANY
gallery image, the image with that custom PID will open, rather than the
one the user clicked on.
@yaquawa
Copy link
Owner

yaquawa commented Feb 12, 2017

As you said, if you use the custom pid PhotoSwipe it won't open the slide for you,
I think this is a bug of PhotoSwipe, so I'd like to postpone your pull request untile PhotoSwope fix this bug.

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