All comments

• 

  Mustafa Yalçın• 40

  @mylcin

  Submitted almost 2 years ago

  I will improve the shadow background later.

  

  News homepage solution with React.js & Tailwind CSS

  1

  30

  

  Lea• 160

  @Hatchino

  Posted almost 2 years ago

  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 !

  0
• 

  Mohammed Abbas• 390

  @Mohammedabbas7

  Submitted about 2 years ago

  The challenge was easy in general but what I found difficult is how to create the layout for a large screen. I used a CSS grid for the layout and I define 12 columns and 12 rows. I just wondering if I did right. Any feedback, please.

  

  Responsive feature section using css grid

  1

  30

  

  Lea• 160

  @Hatchino

  Posted almost 2 years ago

  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

  0

  This comment has received 1 replyShow
• 

  Funsho Ayobanjo• 150

  @ayobanjo

  Submitted about 2 years ago

  I think my media query is a bit shaky when going from desktop to mobile. There were spill outs. How can I handle this better?

  

  Responsive flex layout card

  3

  60

  

  Lea• 160

  @Hatchino

  Posted about 2 years ago

  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

  0
• 

  Vitor riquelme• 30

  @VitorRiquelme

  Submitted about 2 years ago

  Somehow I did, but it's nowhere near the same

  

  Solution

  2

  40

  

  Lea• 160

  @Hatchino

  Posted about 2 years ago

  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)

  1