Responsive layout using flex and grid

Community feedback

• 

  Vitor Lacerda•150

  @vitorlfariaPosted over 3 years ago

  Ficou muito bom Yudi! O responsivo também, ficou bom em todos os viewports que testei. Parabéns!

  1

  ---

  

  Yudi Yoshida•350

  @yudiyoshidaPosted over 3 years ago

  @vitorlfaria Muito obrigado! Bora zerar todos esses desafios! hahaha

  1
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hello, Yudi Yoshida! 👋

  Good job on this challenge! 👏 Your solution looks good and is responsive! 👍

  I suggest,

  • Perhaps adding a max-width to some of the elements of the page (such as the hero text and the review boxes) so that they don't become too wide on extra-large screens (but that's a small thing for the design of this project, really).
  • Taking a look at your solution report and trying to clear up some of the errors that are there. It's generally recommended to include a heading in each <section> or <article> tag you use to identify that element.
  • If you're up for it, adding an intermediary/tablet layout between the desktop and mobile layouts of this challenge would be a nice touch. 😉

  And about your question—I believe using a <link> tag to include fonts for a website is slightly better than using the @import syntax. 🙂

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Yudi Yoshida•350

  @yudiyoshidaPosted over 3 years ago

  Hey @ApplePieGiraffe, thank you so much for your comment!

  Thanks for all your suggestions, I surely will remember them in my next project!

  Keed coding (happy, of course)!

  1