Used Flexbox,Grid and Mobile View First approach

Community feedback

• 

  Travolgi 🍕•31,300

  @denielden

  Posted about 3 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 😁

  22
• 

  Kamasah-Dickson•5,570

  @Kamasah-Dickson

  Posted about 3 years ago

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

  11