qr-code project with flexbox

Community feedback

• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted almost 3 years ago

  Greetings Mortey

  There is work to be done especially with the mobile display, this is caused by having margin-inlines of around 10rems on some elements.

  Marked as helpful

  0

  ---

  

  Mortey Kafui•380

  @MorteyKafuiPosted almost 3 years ago

  @ChamuMutezva thanks for your comment. am yet to work on the responsiveness

  0
• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted almost 3 years ago

  Hiyo,

  It's a good start. I have a few suggestions though:

  1. Your component looks really massive. :D Perhaps could be a bit smaller (you also seem to have a massive padding on it, which isn't necessary).

  2. If you change the width to max-width on your image, it would be responsive too :)

  3. Your title isn't aligned to the center.

  4. Instead of giving your heading and p elements a width, some inline padding would probably serve you better.

  Marked as helpful

  0

  ---

  

  Mortey Kafui•380

  @MorteyKafuiPosted almost 3 years ago

  @FluffyKas thanks for your comment

  0