Responsive landing page using HTML and CSS

Community feedback

• 

  T

  Grace•31,090

  @grace-snowPosted almost 4 years ago

  Hi Seyide,

  Well done on completing your first challenge!

  To improve this project I think you should not use viewport units for font size or pixels so much for paddings. The text is very small on mobile, and things seem off center (although it is hard for me to tell exactly why while looking on my phone)

  The other thing to change are those social links. They should be links really not just icon fonts, and should have a name available to assistive technology to say where each link goes. I would give them a matching width and height too so they stay circular not oval.

  Last thing - add focus states to all interactive elements. That's an important one 😉

  That's all from me, keep going, keep learning and have fun while you do!

  0

  ---

  

  seyide hundeyin•440

  @SeyideHundeyinPosted almost 4 years ago

  @grace-snow Thank you so much for the feedback. I appreciate it!

  0