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

If there is no browser history, exit the slideshow by calling stop. #463

Closed
wants to merge 4 commits into from

Conversation

setnes
Copy link
Contributor

@setnes setnes commented Nov 8, 2015

This is a fix for #460.

@oparoz
Copy link
Contributor

oparoz commented Nov 13, 2015

Thanks, that sounds like a sensible solution :)

Did you test it with various scenarios or just by sharing a Gallery link?

@setnes
Copy link
Contributor Author

setnes commented Nov 30, 2015

I have tried this for awhile, and it seems to be working nicely. What should be tested specifically?

I have tried from an image at the top level of a shared link as well as with images in sub-directories of a shared link. Both work fine. I have tried from a browser that is logged into an OwnCloud account as well as from browsers that are using only the access that the shared link provides... etc.

@oparoz
Copy link
Contributor

oparoz commented Dec 2, 2015

What should be tested specifically?

What you've described + normal use as a logged in user who just browses through his pictures.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants