Blog Preview card

Community feedback

• 

  P

  Kamran Kiani•1,280

  @kaamiikPosted about 1 month ago

  Hi. I have some notes that I wanna mention:

  1. There is no need to use article and section tag here. It's Just a card and can be deployed using a div tag`.
  2. Your img is decorative and does not need any alt text message.
  3. I think Learning is not a button and can be a tag because It takes you to the new page if It is a tag.
  4. You can not use a text inside a div. Try a semantic tag like p.
  5. It's not necessary to wrap everything in a div. Do it when you have to.
  6. Still I see some other problems. Better to revisit your html.
  7. On CSS, You need a good CSS reset. Andy Bell is good.
  8. Your page is not responsive. If you resize the screen size it overflows. Because you restricted your width and height. Never ever do this. You can do this only for some image and logos.
  9. You have to use rem unit for font size and also for spacing you can use em too. and also for using max-width and media query you have to use rem unit.

  0

  ---

  

  Pichika Chandu•110

  @PichikachanduPosted about 1 month ago

  @kaamiik thank you for your clarification iam just a beginner I will rectify those mistakes

  1

  

  P

  Kamran Kiani•1,280

  @kaamiikPosted about 1 month ago

  Yw :) Try to ask more question on discord community If you are hesitate in some part of your code. Mentors can help much better. @Pichikachandu

  0

  

  Pichika Chandu•110

  @PichikachanduPosted about 1 month ago

  @kaamiik of course thank you so much

  0