html and css

Community feedback

• 

  Ridwanullah•210

  @DanonymousCoderPosted over 1 year ago

  Nice work, keep grinding. You just need to add a border-radius to the image. increase the padding of the sub text. And reduce the size of the container holding the image and texts.

  Marked as helpful

  0

  ---

  

  kabir.js•10

  @professorkabirPosted over 1 year ago

  @DanonymousCoder thank you very much, i appreciate.

  0
• 

  Jonathan•20

  @jchu51Posted over 1 year ago

  Hi @professorkabir,

  Great work!

  A few minor suggestions:

  • Address the code formatting, prettier .
  • Using media queries for mobile devices; currently, the QR code image appears too large on mobile screens.

  @media only screen and (max-width: 600px) {
    .className {
      width: ...
    }
  }
  

  • It might be beneficial to use 'rem' or 'em' units for font sizes, padding, and margins.
  • Instead of relying on element selectors, consider using the class attribute more frequently for CSS styling.

  0