@ApplePieGiraffe
Posted
Hello, DuΕ‘an! π
Nice job on this challenge! π Your solution looks good and is responsive! π The slider works pretty well, too! π
I suggest,
- Adding
cursor: pointer
to the slider buttons. - Taking a look at your solution report and trying to clear up the errors that are (looks like you should add some
alt
attributes to the images in the site for accessibility purposes). - There are probably many ways to add transitions to the slider, but one idea is that you can overlay the two images on top of each other and then use JS to change the opacity of each of them when the slider buttons are clicked so that they take turns fade into and out of view. π
Keep coding (and happy coding, too)! π
Oh, and hey, if you found this comment helpful, an upvote would be appreciated! ;)
@DEmanderbag
Posted
@ApplePieGiraffe Hello,
Thank you for the suggestion on your end, I've implemented the first two changes that you suggested, I've must have missed that while I was working on it but that you pointed that out.
The animation is something that I still have to implement but right now I at least have some general idea.
Thanks and happy coding.
@ApplePieGiraffe
Posted
@DEmanderbag
No problem! Just took another look, and the changes you already made look good! π Keep it up and have fun! π