-
Notifications
You must be signed in to change notification settings - Fork 0
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
Image Gallery pagination #264
Comments
@ccheitman I'm going to mark this issue as v1.5, which we'll get to IF there is time at the end. I don't think it's very likely we'll get to it in this version. It's actually a pretty complicated issue, because which picture is "next" depends on the search choices the user made on the previous page, and none of that information is currently being passed though to the individual gallery images. If you see any other websites that handle this gracefully, please let me know so we can use as an example! How I usually handle this as a user is to use control click to open images in new tabs. |
Commenting to note that flickr handles this well. I couldn't find any academic-y sites that did it on a quick glance (except ones that are using javascript to generate the gallery). |
I understand. We did this via Lightbox on the CRA, I believe (@tripodrob ?). But that would be a different technical solution and I agree we don't have time to "change horses midstream." Perhaps it might improve things slightly to open a new window when you click on a thumbnail image? That might make it slightly easier for users to get back to the gallery. |
Yeah, we did use a Lightbox variant on CRA. |
Opening in a new tab / window is an easy solution for us! We could also do lightbox for a single page of search results without too much difficulty, more likely, although for the sake of making sure we finish everything vital I might put that as a 1.5 solution? |
I'm not sure the new tab/window solution would feel right. Might be frustrating to the user? Lightbox for single page of search results is what we did for the CRA. |
I think we can look into that for the future. What I am not sure about re: lightbox is showing all the info alongside the image like we are currently doing. Seems like a lot to fit into an overlay vs a whole other web page. Some testing may be needed. |
This is on hold until we can discuss more thoroughly |
Received an important piece of beta user feedback: We need to have a way for users to page through images instead of having to tack back and forth between an individual image view and the results page.
Also if possible, need a way do a catch-all subject text search.
The text was updated successfully, but these errors were encountered: