@danmlarsen
Posted
Hello there. Congratulations on finishing another challenge! Your solution responds quite well and looks nice 👏
I've made one accessibility observation that might interest you:
- When I hover one of the thumbnails the main image seems to be removed, which is somewhat confusing. I think the problem is somwhere in the updateMainPhoto function. All image urls seems to go to: "https://github.com/35degrees/ecommerce-landing-a11y-fem20/blob/main/images/image-pro"
Happy coding. 💻
Dan
Marked as helpful