Hello everyone!
I found using CSS Grid to be very useful when completing this challenge! Did anyone else do the same or did yall do something different? I would love to know what approach yall used!
p.s. All feedback is welcome!
Hi,
Looking at your design I noticed a few things: -the component is not centered: you can solve this using flexbox: justify-content: center and align-items: center -you did not apply the hovering effect on the buttons: you can look at the design images to see what the page should look like in the active state -you need to fix the border-radius in the mobile version as they are currently messed up
Hi,
Just a quick feedback. Increase the whitespace between your elements. The whitespace is quite important in modern design and your component looks cramped. You can increase the whitespace using padding or margins. Also, the cart button must be at the bottom. You can achieve this by applying some margins, which will also solve your whitespace issues. However, I would rather recommend justify-content: space-between.
Good work and keep on coding
please send advice for me
Hi,
Just a few things that I noticed at a first glance: -you did not apply border-radius to the overlay. Try to apply the same border radius as your image. -try to delay the overlay when hovering. -the difference between the border-radius of the image and the whole component does not seem very aesthetically pleasing. Try to apply the same border-radius and see if you are happy with the results.
Good work anyway
I had to rush building this, I'm really eager to finish my downloaded projects and move on to working with the API projects here.
I just added the Intersection observer API to add some cool animations, let me know what you think about it
Hi. This looks very nice. However, a few things that I noticed. Try to go with a less aggressive shadow on the nav bar, double-check the colors of the nav bar links, your first article image is larger than the other ones, and try to increase the padding on the footer. Well done anyway!
