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

drag to close image to thumbnail #1431

Open
osemec opened this issue Oct 7, 2017 · 7 comments
Open

drag to close image to thumbnail #1431

osemec opened this issue Oct 7, 2017 · 7 comments

Comments

@osemec
Copy link

osemec commented Oct 7, 2017

With showHideOpacity = true and getThumbBoundsFn = false, when I drag an image with mouse (up or down) to close it, the (full res) image jumped to center for a short time, before it closes.
(I'm using square thumbnails and full res images with different res.)

@osemec osemec changed the title drag close image to thumbnail drag to close image to thumbnail Oct 7, 2017
@arnowelzel
Copy link

I'm not sure, if this is what you're talking about - but I noticed that closing an image by dragging it upwards or downwards out of the view, it will not just be closed, but the whole image will show again and then fade out - which can be a bit irritating.

When dragging pictures up or down to close them, the whole fading stuff should not appear at all, since the picture is then already out sight.

@osemec
Copy link
Author

osemec commented Nov 23, 2017

Yes, this is the bug I was talking about.

@jefferyto
Copy link

Please give my PR #1563 a try - I believe it is a nicer-looking fix for this issue :-)

@arnowelzel
Copy link

I believe, PhotoSwipe 4 does not get any more updates here - the last commit is about 11 months ago and there are 60 open pull requests.

About your proposed fix: I'll try that on my own fork.

@arnowelzel
Copy link

Ok, looks nice. Now the picture does not just disappear but fade out properly. I will use this in my fork and for my WordPress plugin as well.

@arnowelzel
Copy link

JFTR: #1528

@jefferyto
Copy link

Thanks - I'm not expecting my pull request to be merged, just thought there may be other users who can benefit from it while waiting for the next version to be released.

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

No branches or pull requests

3 participants