Used Flexbox,Grid and Mobile View First approach

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

  Hey Nazim, congratulations on completing the challenge! You did a great job 😉

  Let me give you some little tips for optimizing your code:

  • Tip of graphic design: with font-family:" Big Shoulders Display ", cursive the browser will use the Comics Sans font when it doesn't find the first font indicated (you can seen during loading)... for the designer it's a really awful font! I would rather replace it with a font-family:" Big Shoulders Display ", sans-serif much more similar to the primary font.
  • add descriptive text in the alt attribute of the images
  • to avoid the jump that occurs when you mouse over the button, assign the same border to the button even in the normal state
  • add transition on the element with hover effect
  • instead of using px or % use relative units of measurement like rem -> read here

  Hope this help! Happy coding 😁

  2

  ---

  

  Nazim Filzer•110

  @NazimFilzerPosted over 2 years ago

  @denielden Tysm for Giving ur valuable response.. One doubt was that should i use rem unit everywhere?

  1

  

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

  @NazimFilzer You are welcome!

  The more you use them the better ... but each size has its own peculiarities and consequently its correct and suitable use

  1
• 

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted over 2 years ago

  Your solution looks great on mobile. Great job there, keep coding👍

  1

  ---

  

  Nazim Filzer•110

  @NazimFilzerPosted over 2 years ago

  @Kamasah-Dickson I had a doubt. In which all methods can we score points in Frontend mentor

  0