Responsive landing page using CSS flexbox

Community feedback

• 

  Alex Kim•1,325

  @alex-kim-devPosted over 4 years ago

  Pretty decent job! I like your gulp setup and the way you used BEM. Also the page matches the design very well! There're just 2 small things that i would add:

  1. styling for controls :focus state
  2. rel="noreferrer" attribute for the external links, here is more info

  0

  ---

  

  Ksenia•1,350

  @kseniusPosted over 4 years ago

  @Alex-K1m Thank you for your suggestions, I'll definitely take them into account.

  0
• 

  Safa Ruknuddin•385

  @safrukPosted over 4 years ago

  Really great job! I have noticed you haven't posted in a while. I love to look at your solutions. Really neat sass outlay

  0
• 

  Sodiq•100

  @salsod-starPosted almost 5 years ago

  Well-done

  0
• 

  Adarsh Pratap•5,515

  @adarshcodesPosted almost 5 years ago

  It's really amazing 😍😍😍 Nearly same. Great job ksenia

  0

  ---

  

  Ksenia•1,350

  @kseniusPosted almost 5 years ago

  @adarshcodes Thanks!

  0
• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  Amazing work, Ksenia. You've done a really good job at replicating the design and your code looks great! The only small thing left to do is to try resolving the accessibility issues in the report.

  Keep up the great work!

  0

  ---

  

  Ksenia•1,350

  @kseniusPosted almost 5 years ago

  @mattstuddert The issues have been resolved :) Guess I need to pay more attention to accessibility. Thank you for the feedback, I really appreciate it!

  0