-
-
Notifications
You must be signed in to change notification settings - Fork 531
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
feat: fullscreen images slideshow #1120
base: main
Are you sure you want to change the base?
Conversation
Great to see this! Is it ready for testing in demo? Make sure you push the static files as well!!! |
@mgogoulos i still need to clean up the code a bit test it more and add keyboard keys functionalities |
This looks great so far! I've made these two changes and pushed them in main, can you sync in order to get them?
It expects a setting SLIDESHOW_ITEMS and will read that number, otherwise it sets a default of 30. Other than that, I have the following suggestions:
|
Description
Steps
Pre-deploy
Post-deploy