All comments

• 

  m•140

  @mhz-777Submitted about 2 years ago

  Hey everyone just finished this project.

  Let me know what you think and where I can improve!

  Thanks :)

  

  Product Preview Card

  1

  40
  

  Paula Rodrigues•250

  @paulaabroPosted about 2 years ago

  Nice job!

  • You got an HTML issue because your section with the images doesn't contains a h2-h6 so I would recommend changing that.
  • Also, something that was recommended to me in the beginning and I found awesome is to use the picture tag it works like this:

  <picture>
    <source media("min-width: 600px") srcset="desktop.png">
    <img src="mobile.png">
  </picture>
  

  To learn more see de DOC Responsive Images

  Marked as helpful

  0
• 

  Eduardo Alexandre•30

  @DuAlexandreSubmitted about 2 years ago

  

  QR-Code-Component-HTML/CSS/FlexBox

  3

  20
  

  Paula Rodrigues•250

  @paulaabroPosted about 2 years ago

  Congrats on the solution 🎉

  Feedback time

  • I would recommend using a reset.css for reseting the css of the browser so you get more control over it. I use RESET CSS 2022 but you may prefer RESET CSS.
  • It's not back practice but I like to remove the unit when its 0 so:

  margin: 0px;
  # becomes
  margin: 0;
  

  Nice job, keep it up!

  0