All comments

• 

  Mohammad-Nayem• 20

  @nayem13913

  Submitted over 1 year ago

  

  Created a Qr code component

  1

  50

  

  Irfan RIzki Saputra• 260

  @nafri97

  Posted over 1 year ago

  great bro! but I recoment use center instead of using justify .div h1{text-align: center;} it will more suitable with .div p{} which already nice with center-aligned.

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  ruthermadz• 10

  @ruthermadz

  Submitted over 1 year ago

  Updated solutions. Changes made:

  • use merge query
  • adjust margins and padding
  • Add responsiveness to font-size

  

  First Frontend Mentor Project using Basic CSS

  1

  20

  

  Irfan RIzki Saputra• 260

  @nafri97

  Posted over 1 year ago

  I'm still beginer too, but I have some suggestion:

  1. try to use responsive page, add @media screen and(max-width: 375px){.card{width: 80%;padding:5%}} , it means this code is executed when shown iresolution =<375.
  2. add margin-top instead of just using margin: auto (you could combine them both) to add more space above card.
  3. in mobile responsive try to avoid using font with vh, because smaller the screen, it will less visible to read. sorry I still don't know much, but let's learn together and happy coding :)

  Marked as helpful

  0