Responsive landing page using flexbox

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted about 4 years ago

  Hey Safa, I've not got much to say other than you've done an amazing job! The one area I'd say that could be improved is how you're using headings in your HTML.

  At the moment, you've got h1, h3, h4, and h6 headings on this page, but no h2 or h5. I'd recommend avoiding skipping heading levels unnecessarily. This can cause accessibility issues in the content hierarchy for screen reader users. You're also using a h6 for the introductory paragraph ("Take your financial life online..."). This is a paragraph of text, not a heading, so a p tag would be best here.

  Some re-working of the headings and this would be near perfect. Awesome work! 🙌

  2

  ---

  

  Safa Ruknuddin•385

  @safrukPosted about 4 years ago

  Thanks Matt!! It's always great to hear from you. I never knew that skipping heading levels is an issue and keep using h6 just out of habit. Will remember to keep this bit in mind henceforth. Thanks!!

  0

  

  T

  Matt Studdert•13,611

  @mattstuddertPosted about 4 years ago

  @safruk you're welcome! Here's a great article for more information about page structure with headings: https://www.w3.org/WAI/tutorials/page-structure/headings/.

  1
• 

  Adegbulugbe-Israel•15

  @Adegbulugbe-IsraelPosted about 4 years ago

  Great job man!

  1

  ---

  

  Safa Ruknuddin•385

  @safrukPosted about 4 years ago

  Thanks!! :)

  0