Skip to content
This repository has been archived by the owner on Nov 14, 2018. It is now read-only.

[Imageviewer] add gallery functionality #1111

Closed
wants to merge 4 commits into from

Conversation

sualko
Copy link

@sualko sualko commented May 17, 2013

Show buttons for previous/next image.

@ghost
Copy link

ghost commented May 17, 2013

Can one of the admins verify this patch?

@karlitschek
Copy link
Contributor

@butonic @icewind1991

@butonic
Copy link
Contributor

butonic commented May 17, 2013

If I remember correctly we wanted to reuse the image galleys slideshow to preview images in the files app. I have this working for a customer but did not yet find the time to backport it. @jancborchardt whats your opinion on the general approach? I'd rather only have one image preview implementation.

@butonic
Copy link
Contributor

butonic commented May 17, 2013

@sualko there is also still an alert() in the PR

@jancborchardt
Copy link
Contributor

@sualko I assume @butonic’s pull request #1123 superseded this one here. @butonic is that correct and can we close this? @sualko sorry for the miscommunication. :)

@jancborchardt
Copy link
Contributor

Closing this because it was superseded by #1123 implemented by @butonic, see my previous comment.

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

Successfully merging this pull request may close these issues.

None yet

4 participants