Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update index.js #30

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update index.js #30

wants to merge 1 commit into from

Conversation

benstraw
Copy link

added optional click navigation which will only show up if set:

[params.gallerydeluxe]
enable_click_navigation = true

added optional click navigation which will only show up if set:

[params.gallerydeluxe]
        enable_click_navigation = true
Copy link

netlify bot commented Aug 23, 2024

Deploy Preview for gallerydeluxepreview ready!

Name Link
🔨 Latest commit f65b64a
🔍 Latest deploy log https://app.netlify.com/sites/gallerydeluxepreview/deploys/66c835482bd3dd00089438ab
😎 Deploy Preview https://deploy-preview-30--gallerydeluxepreview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@benstraw
Copy link
Author

you may view the change with that configuration value set here - https://benstraw.com/categories/photography

@benstraw
Copy link
Author

Ooops, I just realized I reversed the swipe direction calls on my link buttons. It just needs to be:

leftNav.addEventListener('click', function () {
				swipe('right');
			});

and

rightNav.addEventListener('click', function () {
				swipe('left');
			});

@bep
Copy link
Owner

bep commented Aug 23, 2024

I'll have a look at this later, but a little note: If this was my site I would add compress/reduce the size of the full size images. 5MB JPEGs seems a little on the big side for the web.

@benstraw
Copy link
Author

benstraw commented Aug 23, 2024 via email

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants