All comments

• 

  Elouka972•80

  @Elouka972Submitted over 1 year ago

  Heey !

  Do you have any advices to give about my code?

  

  Results-summary-component-main

  1

  50
  

  P

  RogelioCamargo•200

  @RogelioCamargoPosted over 1 year ago

  Your solution looks awesome. I do recommend this methodology for naming class names: BEM 101.

  I recommend picking up this naming scheme for larger projects as it'll make your CSS writing more maintainable!

  Marked as helpful

  0
• 

  Juan Pablo Favale•10

  @juanpablofavaleSubmitted over 1 year ago

  

  CSS Flexbox, HTML, CSS

  2

  50
  

  P

  RogelioCamargo•200

  @RogelioCamargoPosted over 1 year ago

  Your solution is awesome and looks great on all devices.

  What I noticed:

  1. Font is incorrect. In case you don't know how to use Google fonts, this is a good resource: How to use Google Fonts
  2. Use "min-height: 100vh" for body instead of "height: 100vh." Setting the height: 100vh may result in the component being cut off on smaller screens.

  0
• 

  the-coding-beekeeper•270

  @the-coding-beekeeperSubmitted over 1 year ago

  Hi folks, this is my first challenge on frontend mentor. I look forward to any feedback, as I want to improve my coding skills.

  Greetings!

  

  QR-code integration, plain html &css

  2

  50
  

  P

  RogelioCamargo•200

  @RogelioCamargoPosted over 1 year ago

  I love that you used CSS variables. It looks like your solution isn't responsive. I personally suggest taking a mobile-first approach.

  Mobile First Design

  Marked as helpful

  1