HTML, SCSS, Vanilla JS

Community feedback

• 

  Gerben Dol•3,115

  @GerbenDolPosted over 4 years ago

  Hey! I think you did really well! There's a few changes I'd make:

  • Make sure the background image on your body only covers the part that doesn't have the photo on it. It's scaled very largely now.
  • Remove the border from your button
  • Make sure the background behind the logo is white on mobile

  All in all it's looking great, so keep up the good work! 💪🏻

  0

  ---

  

  Md Zeeshan•80

  @AsilTalyaniPosted over 4 years ago

  @GerbenDol Thanks for your feedback it means a lot, I am looking into it and I will update as soon as possible. One more question, should I just update my repository, frontend mentor fetch that updated project or I have to re-upload it?

  0

  

  Gerben Dol•3,115

  @GerbenDolPosted over 4 years ago

  @AsilTalyani No problem, take your time! Glad to hear it's helpful.

  Just update the repo and make sure your live location also updated, which should be automatic.

  The platform will just show the updated versions, so no worries. 😁

  0