Responsive page using Sass

Community feedback

• 

  Bonrey•1,130

  @Bonrey

  Posted about 4 years ago

  Hello, @olgak169! 👋 Fine effort on this one!

  I like how your site is almost pixel perfect. I know how hard it is without any Sketch or Figma designs, so congrats on that! Also, it's great that you put in some time to add transitions and form validation. In my opinion, all this visual stuff is always worth the effort. 😊

  There're a few little things you could fix to make your site look even better:

  • Make your logos clickable. It'd be nicer if the user could move to the top of the page (or refresh it) on the Huddle logo click.
  • Consider removing one of your h1 tags: there should be at most one such tag for a page. (You can also check your HTML Validations issue for more info.)
  • Add some transition-duration to your links and buttons. That way, the overall look will be smoother.

  In addition, it's great to see that you used the :invalid selector. To be candid, I didn't know it existed before I checked your code. 😅

  It's great to see you're completing many challenges on this site. Keep on coding! And best of luck with everything you're learning! 😄

  02