Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

E-commerce Product Page using Astro, Sass and Vanilla JavaScript

P
Kamran Kiani 1,040

@kaamiik

Desktop design screenshot for the E-commerce product page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
3intermediate
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

I am most proud of my efforts to ensure accessibility in my project, particularly in creating a responsive navbar and using dialog elements effectively. Next time, I would definitely use a framework to streamline the development process.

What challenges did you encounter, and how did you overcome them?

One of the main challenges was ensuring that the lightbox and cart dialogs were both functional and accessible. I had to pay close attention to various aspects in JavaScript to ensure necessary updates. I overcame these challenges by meticulously adhering to accessibility guidelines and thoroughly testing my code.

What specific areas of your project would you like help with?

I would like help with ensuring that my implementation of the lightbox and cart dialogs is as efficient and accessible as possible. Additionally, any feedback on improving the overall accessibility and responsiveness of the navbar would be greatly appreciated.

Community feedback

Wendy 1,970

@wendyhamel

Posted

Hi there!

Nice solution with just some minor layout issues I'd like to point out.

Your menu items are not visible on desktop. Looks like .nav__content: opacity: 0 is the problem.

The thumbnails are not square in your carousel.

Your Javascript looks structured and the page is responsive and acceptably accessible.

Happy coding!

Marked as helpful

0

P
Kamran Kiani 1,040

@kaamiik

Posted

Thanks @wendyhamel It was because of prefers-reduced-motion that did not show the nav items. And also I reduce the thumbnail buttons gap to make them more like a square.

0
Wendy 1,970

@wendyhamel

Posted

@kaamiik ah! Now I am curious to see it without prefers-reduced-motion.

I will take a look!

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord