Implemeted using flex-box (last commit) and inline-block (prev commit)

Community feedback

• 

  RaffayElahi•130

  @RaffayElahiPosted over 1 year ago

  Hey @younasBnHasainar! I saw your code, it was great but some practices will greatly help you in your long-term web development journey. Those practices are:-

  • I saw that your HTML wasn't structured as you were using randomly h2 and h4 tags, try using heading tags according to the importance of heading and heading number. This practice will make your SEO friendly also use section tags for covering your whole main content instead of div. These look like small negligible mistakes but these make difference in readibility and access to your code.
  • Second, you could make the whole body a flex container in order to center out the main div instead of using out margins.
  • Try using ul and li for listing out things instead of listing out things in p tags. Also try to use flex containers as much as possible to make your content aligning out either in row or column or wither centering it or giving out space evenly. Flexbox is really helpful.

  Hope these suggestion were helpful. Happing coding and regards.

  0