All comments

• 

  Elouka972• 80

  @Elouka972

  Submitted over 1 year ago

  Heey !

  Do you have any advices to give about my code?

  

  Results-summary-component-main

  1

  50

  

  P

  RogelioCamargo• 200

  @RogelioCamargo

  Posted 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

  @juanpablofavale

  Submitted over 1 year ago

  

  CSS Flexbox, HTML, CSS

  2

  50

  

  P

  RogelioCamargo• 200

  @RogelioCamargo

  Posted 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-beekeeper

  Submitted 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

  @RogelioCamargo

  Posted 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