Responsive landing page using css flex

Community feedback

• 

  Suzunatsu•120

  @st0272Posted 21 days ago

  Your code is generally well-structured, but several issues and areas for improvement in terms of semantics, accessibility, and organization. Below is my opinion for improvement.

  Use of <p> for titles

  The <p> tag is used for headings like "Reliable, efficient delivery" and "Powered by Technology", which should be in <h1> and <h2> respectively for better semantics.

  Missing alt attributes in images

  The <img> tags have an empty alt="", which is not good for accessibility.

  Improper nesting of divs

  The content-cards-two div is nested inside the .content-cards but is structured inconsistently. It appears that all .card elements should be direct children of .content-cards for better alignment and flexibility. Using grid to style this four-card layout is efficient.

  I hope my advice will be helpful for your coding. Keep up the good work!

  0