@mylcin
Latest comments
- @Hatchino
Hi, I just want to leave some feedback with aesthetics details. First I think that is missing pointer on the icon menu. And I don't know how Tailwind works but button tag is better than img for accessibility. For the second section with news, you should add padding over the title (h3) and then delete the padding below the last p to center all the content. Happy coding !
- @Mohammedabbas7@Hatchino
Hi, Why are you used 12 columns and 12 rows, I think 3 columns and 4 rows is enough. You define the grid-template-areas and all is working. You can look at my solution if it can help you. In your code I think it's unintentional but you added a class to your span which is undefined. Whatever, I like hover transition on the footer.
Marked as helpful - @ayobanjo@Hatchino
Hi. First, the r of "perfume" is missing. It also lacks the letter-spacing property which allows you the spacing between lines for your paragraph. The icon is not the same as on the model. Your responsive is not adjusted between 440px and 374px (the price exceeds the card). Adding styling to the attribution is just my opinion.
Marked as helpful - @VitorRiquelme@Hatchino
Hi, The active part is missing. When you navigate with your mouse, the texts colors change and there is an interaction with the image. The responsive part is missing. Icons are not centered (use flexbox). You added a box-shadow that does not exist on the model. And you must have noticed that the colors are not similar (I'm having trouble too, don't worry)
