Huddle landing page with a single introductory section

Community feedback

• 

  Elroy Toscano•630

  @elroytoscanoPosted almost 3 years ago

  Hi Catherine, good job with the design and not getting any Accesibility and HTML issues.

  Here are some areas you could improve:

  1. The original design has a padding of certain units all around(maybe 6 or 8rem), the positioning of the social media icons breaks that alignment.
  2. In case you're making a circular button, in the styling the width and height should be the same. The facebook icon is elliptical, after checking through your code, the media-icon class should have the same height and width,(32px -width and 38px height is causing the elliptical structure), whereas twitter and insta icons are perfect.
  3. When hovering over any button/link, cursor:pointer and subtle change in backgroundColor provide a good visual feedback to the users.

  Hope this helps.

  3

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 3 years ago

  @elroytoscano Thank you for the feedback, much appreciated!

  0
• 

  ROCKY BARUA•1,090

  @DrougnovPosted almost 3 years ago

  Hi @catherineisonline, great job with the layout and responsiveness.

  However, here's something you may consider:

  -The background pattern isn't flexible enough. Use background-size: cover to fix it. -The Facebook icon doesn't look good. Add some padding to both sides.

  Overall, great solution. Happy coding :)

  1

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 3 years ago

  @Drougnov Thank you!

  0
• 

  Zein•360

  @Zein-MBPosted almost 3 years ago

  It would be more softer if you put transition 0.3s to the button Good job by the way!

  0

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 3 years ago

  @Zein-MB thanks for the suggestion!

  0
• 

  BadT•230

  @saodinhPosted almost 3 years ago

  You did a great job, though some things can be improved. Since the "Register" button is actually a link to the register page and doesn't belong in a form , you should use <a> tag for it. The three social icons should be wrapped in <a> tags as well (because they are also links to the social media pages).

  0

  ---

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 3 years ago

  @saodinh thank you for the feedback!

  0