@SabrinaY123
Iulian Cosmin Bruma
@brumacosminAll comments
- @brumacosmin
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
- also, in the mobile version the whitespace between the text and the buttons should decrease.
Marked as helpful - @JezzeX@brumacosmin
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
Marked as helpful - @AhmedAbdullah11@brumacosmin
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
Marked as helpful - @abdullah43577@brumacosmin
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!
