Huddle landing page

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted almost 4 years ago

  Hi

  This looks really nice on mobile, well done!

  I'd lose the div wrapping the button and place classes on the button itself instead (really, that element would probably be a link to a registration page rather than a button, but that's not that important to change)

  I would definitely add links to your social icons though ☺

  Good job on this

  0

  ---

  

  Gehad Salem Fekry•50

  @Gehad75Posted almost 4 years ago

  @grace-snow Thank you a lottt <3. I will change this :)

  1
• 

  Web Frog•1,840

  @MasterDev333Posted almost 4 years ago

  Great work @Gehad. It would be better if you add some transitions to your buttons. transition: all .3s ease-in-out; Add more media queries to make it fully responsive, especially under 1200px. Hope it helps. Happy coding~ :)

  0

  ---

  

  Gehad Salem Fekry•50

  @Gehad75Posted almost 4 years ago

  @MasterDev333 Thank you a lot for your notes, I work on doing this <3.

  0
• 

  Aleksandr Aksenov•750

  @axevldkPosted almost 4 years ago

  Hi, @Gehad Salem Fekry ~ I have studied your work, and good work on your first challenge. Here are some of my opinions on your work.

  1. Font sizes don't match with design. I think you can revise them easily.

  2. Hope you to add padding to container, so that it won't fit whole screen under small resolution.

  3. You can set width of image as 50% or so, so that it won't take too much space compared to content.

  Hope my words can help you even a bit. Happy coding ~ ✨

  0

  ---

  

  Gehad Salem Fekry•50

  @Gehad75Posted almost 4 years ago

  @axevldk Thank you a lot for your notes, I will update the code. <3

  1

  

  Gehad Salem Fekry•50

  @Gehad75Posted almost 4 years ago

  @axevldk I have made the changes but they aren't appearing here in the comparison design :(

  0