Responsive article preview component using grid

Community feedback

• 

  Basant Soni•450

  @SoniBasantPosted over 1 year ago

  • Don't use % with width. % is used when you write it in CSS. In HTML, % is not required. I suggest to remove it completely and handle style in CSS.
  • Change .article_title to h1 instead of p, so that you can have at least one mandatory heading. Change font-size as you need.
  • In first para, Shift the overall..., First sentence should end with by and second with to.
  • You choose wrong color for date-month-year. You use color: hsl(214, 17%, 51%); I think you should use color: hsl(212, 23%, 69%);
  • No or very light box-shadow with image. Check with the image you downloaded or in the comparison given above. It is only with content.

  Placing of the image of person and name is good. Other paddings are also good. Overall card is looking great.

  Good Job. : )

  0