Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Buttons on home page #239

Closed
abisummers opened this issue Oct 8, 2018 · 3 comments
Closed

Buttons on home page #239

abisummers opened this issue Oct 8, 2018 · 3 comments

Comments

@abisummers
Copy link
Contributor

When you click the next button on the home page (src/home.html), the picture changes (after the second click) but makes all other pictures disappear. When you click the previous button, nothing happens. On the home.js getElementById("img") is used in the displayNextImage function but I think that should be different since img is never used as a ID.

I'm going to try and fix them if I have time, but they're open for whoever wants to try to fix them!
@sudz123
Copy link
Owner

sudz123 commented Oct 8, 2018

I have referenced this in #212 . Do take a look at this and the other issues I have addressed there. I am soon going to lock the appearance of the front end in home.html, these are the last remaining changes I want. What would be great is that next button and prev button fast-forward the scrolling in the respective directions.

@abisummers
Copy link
Contributor Author

I'll take a look tomorrow morning. Thanks for hosting this project!!

@sudz123
Copy link
Owner

sudz123 commented Oct 9, 2018

This has been fixed lately. Closing this.

@sudz123 sudz123 closed this as completed Oct 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sudz123 @abisummers