Easybank Landing Page

Community feedback

• 

  mstanka•360

  @mstankaPosted over 4 years ago

  Hi, you did a great job with this challenge. Your code is well structured and easy to follow. I learned from your code that I can use 'setAttribute' to change the icon for mobile navigation. I miss this possibility somehow. So, thanks for it. And keep coding :)

  1

  ---

  

  Samaila Philemon Bala•180

  @sa-maPosted over 4 years ago

  Thank you very much

  0
• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted over 4 years ago

  Awesome work, Samaila! You've done a really good job on this challenge 👍

  Have you ever tried using min-width media queries instead of max-width? It's quite a common workflow with front-end developers to use them and work mobile-first. It can often lead to less CSS code and has the benefit of loading in fewer styles for mobile users, which can be a nice performance gain.

  Keep it up! 🥳

  0

  ---

  

  Samaila Philemon Bala•180

  @sa-maPosted over 4 years ago

  Thank you Matt, I'll try that for my next project @mattstuddert

  0