-
Notifications
You must be signed in to change notification settings - Fork 4
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
Test new media viewer #50
Comments
@kimisgold I'm afraid I may have gone wrong somewhere but am not sure where. I went into the theme then did git checkout lightgallery and it says it switched. When I click through any items on the front end I'm getting this screen: I checked and it seems I do have the error logging turned on but the log is empty. Cleared cache/did hard refresh nothing changed. |
@kimisgold here's the error: Omeka has encountered an error Zend_Http_Client_Exception
Zend_Http_Client_Exception: Passed parameter is not a valid HTTP URI. in /var/www/html/katknow/OmekaTesting/application/libraries/Zend/Http/Client.php:318 |
This seems to be related to Scripto. Can you deactivate it and see if you're still getting the error? |
@kimisgold Just deactivated and seems to be working, thanks! |
@kimisgold I'm able to get full page view on Chrome, Safari, Firefox, and Edge. However, I cannot get the zoom in/out feature to work on Safari. I can only get the download button to appear for videos on Chrome and in Edge (chrome tested both Mac & PC). I can't seem to get the thumbnail pager to appear, would that just be in the regular browse page or would I need to look in a different area? Hard refreshed and cleared cookies/cache on all browsers (know it did something because that's how I got the download button to appear for Chrome & Edge). Not having much luck with any of it on mobile browsers, but going to update my phone and see if that helps. |
@kimisgold seems to work well across browsers on both desktop and mobile. Doesn't seem to have the same nav bar issues, but I do have to switch back to the master branch from new-admin-theme in order to add sub navigation to test. Don't know if that was somewhere in the slack thread and I missed it. |
There was a navigation bug that I fixed just this morning. If you pull the latest changes on |
@kimisgold Yep, that was it! Everything looks good. |
I've just implemented the lightgallery jquery media viewer. It includes the following features:
It lives on the
lightgallery
branch and needs to be tested on mobile and desktop browsers. I'm also open to feature requests in feedback.The text was updated successfully, but these errors were encountered: