All comments

• 

  Shin Hijikata• 110

  @shwebdeveloper

  Submitted about 1 year ago

  

  QR Code Component using HTML & CSS

  1

  50

  

  Brine• 140

  @brinereef1

  Posted about 1 year ago

  Congrats on successfully completing this project

  You did a good job and it's looks really great

  Just a small suggestion, you can use media query at max-width 401px to change the margin of h1 and p inside article element to margin-block, it will avoid taking extra white space

  @media only screen and (max-width: 401px) {
  h1 {
  margin-block: 1.5625rem
  }
  
  p {
  margin-block: 0.875rem;
  }
  }
  

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  キノシタ• 10

  @Diavolo01

  Submitted over 1 year ago

  

  Building QR-code with HTML and CSS

  2

  50

  

  Brine• 140

  @brinereef1

  Posted over 1 year ago

  Congrats on complete your first project.

  I have one suggestion, your design is breaking on 375px width device

  So you can try both min and max media screen to 375px.

  @media only screen and (max-width: 375px) 
  @media only screen and (min-width: 375px)
  

  Apart from that your solution is great 👍

  Good luck

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  mateusceci-coder• 70

  @mateusceci-coder

  Submitted over 1 year ago

  It´s my first page that i submit, im really new, so any advice or suggestions will be great!

  

  Beginner page using HTML and CSS

  3

  40

  

  Brine• 140

  @brinereef1

  Posted over 1 year ago

  It looks great, only one suggestion to align the icon with the word

  you can use

  <div class="skill reaction">
  <div>
  <img src="./images/images/icon-reaction.svg" />  
  <p>Reaction</p>
  </div>
  </div>
  

  for all your four skills and in the css file

  .skill div {
  display: flex;
  align-items: center;
  }
  

  1

  This comment has received 1 replyShow