@kwalker3000
Posted
Hello, Jose.
Nice solution.
Regarding the carousel, I do feel like there is a bit too much white space, and I see you are creating the effect by shifting the <ul> container over with each click. What if instead of shifting the <ul> you simply switched out images? This shouldn't require much except a little javascript and a loop.
I also see you have a couple of accessibility issues regarding your carousel buttons. I usually fix this by adding an aria-label, but there are other ways. Here is a nice article on accessible icon buttons
The site's responsive behavior is great so is the overall spacing.
Hope this was helpful. Wish you great success on your journey.
Marked as helpful
@jchapar
Posted
@kwalker3000 Really appreciate this Kevin! I'll be sure to go back and adjust!
