Responsive layout with flexbox

Community feedback

• 

  Psalmorg•190

  @PsalmorgPosted 13 days ago

  This is good but it can better and i think:

  • physically i think your font-size is too big and it need to be reduced;
  • the right tag for each link is anchor tag since we know they navigate to other web and styling them block will remove the; inline property the tag possess and a little bit of top and bottom padding will make it look nice;
  • the <p> can be use for instead of <span>. I learn recently that there is <address> tag itself that can do that since it contain address;
  • reducing the width be good also.

  Marked as helpful

  1

  ---

  

  Yosef shawah•30

  @yosefshawahPosted 13 days ago

  @Psalmorg thanks for the feedback much appreciated!

  0