Handling JavaScript ! , not watching any tutorial and figuring it out by my self ! :)
What challenges did you encounter, and how did you overcome them?Working with the Json Data was the most difficult challange so far. Used fetch .
What specific areas of your project would you like help with?Help me write cleaner code ! Thanks (;
Hey, awesome job on using CSS variables! 🎉 It's great to see you applying them so well it makes your code cleaner and more maintainable. Keep up the good work, you're definitely getting the hang of this! Just a few small tweaks here and there, and you'll be on your way to writing even more efficient code. Well done!
Main, Section, Footer) is best practice For better SEO and readability.alt attribute in image tag should not be empty. It is important for accessibility and SEO.Just appreciating your job, no comment
This first time using JS so I spent a lot of time.
What specific areas of your project would you like help with?Any best practices for using JS.
Hey good work, you're definitely getting the hang of this! Just a few small tweaks here and there, and you'll be on your way to writing even more efficient code. Well done!
I am proud of the use of grid area, which makes the layout responsive and looks great.
Next time when designing a grid I will start with grid area, which makes the layout faster
What challenges did you encounter, and how did you overcome them?I had a problem with the grid but after reading a few guides I discovered the grid area feature which makes the cards look even and nice
What specific areas of your project would you like help with?I'm asking for general advice on design, code readability, and responsiveness.
Hey, awesome job on using CSS variables! 🎉 It's great to see you applying them so well it makes your code cleaner and more maintainable. Keep up the good work, you're definitely getting the hang of this! Just a few small tweaks here and there, and you'll be on your way to writing even more efficient code. Well done!
styles.css in its own folder.main and footer improves your site’s readability, accessibility, and SEO-friendliness.picture, video, canvas, svg, input, button, textarea, select, h2, h3, h4, h5 and h6 there are no such elements.Hey, awesome job on using CSS variables! It's great to see you applying them so well it makes your code cleaner and more maintainable. Keep up the good work, you're definitely getting the hang of this! Just a few small tweaks here and there, and you'll be on your way to writing even more efficient code. Well done!
styles.css in its own folder.header, main, and footer improves your site’s readability, accessibility, and SEO-friendliness.width for the body element since it fills the viewport by default.#f2f2f2 is being used as the background-color, but it's not part of the design.border-top: 3px solid in .card and apply a different color for each card.I am most proud of the responsiveness of the page.
What challenges did you encounter, and how did you overcome them?One of the challenges I encountered was the difference in the mobile view and desktop view images. I used the visibility:hidden property to switch the images between screen sizes.
Another challenge was getting the image to fit within the image-container. I used em units and this worked.
Another challenge I faced was adjusting the width of the desktop view.
What specific areas of your project would you like help with?I would like feedback on best practices to achieve responsiveness on this component.
You’ve done a great job! I love that you used semantic HTML elements it’s awesome for both readability and SEO optimization.
In the HTML section, the main element is used as a card, but there should be an additional div to act as the product container. For images, it's better to use a single img element and apply styling for various screen sizes as needed. Also, the 'Add to cart' element should be a button, not a div. Additionally, it would be more appropriate to wrap the container div with the class attribution in a footer element.
For CSS, using 62% for the font size is too large, it’s recommended to use units like px for better control. The property display: flex on the body element is unnecessary. Instead, apply flex to the main element if you're centering the product card. To make the code more maintainable, consider using CSS variables for colors instead of hard coding them multiple times, which will make updates easier. Lastly, you've only implemented a media query for one screen size, but it would enhance responsiveness if you added queries for screen sizes like 768px, 1024px, 1140px, and more.
Lastly, it's not best practice to place the CSS file in the assets folder, as it's a core part of your web app's program files and not just an asset.
