Huddle landing page w/ a single introductory section - Responsive

Community feedback

• 

  Cody Lewis Zites•570

  @zitescodyPosted almost 3 years ago

  Hello, @NewCoderEx! Welcome to the Frontend Mentor community! Good job on this. Below is my feedback.

  1.) Be sure to nest your content within landmarks. I would place a <main> landmark before your <img> logo tag and after the closing `</div>' element that surrounds all of your main content. This gives your code semantic meaning and allows for screen readers to parse your code easier. It is a simple yet profound fix!

  2.) I wouldn't nest your <button> tag inside an <a> tag. Ideally, the <button> tag will be nested within the <form> element that will redirect to another url when the button is pressed. For the purposes of this project, you do not have to worry about that. I have attached an article below to help you clarify how to use buttons.

  That's it! Keep up the good work. I challenge you to incorporate CSS custom properties into your CSS on your next project. I have also linked an article that explains how to use them. They are so helpful; making your css much more simple.

  Buttons: Check out Buttons Here

  CSS Custom Properties: Check out CSS Custom Properties here

  Marked as helpful

  1
• 

  Mark Colinares•60

  @NewCoderExPosted almost 3 years ago

  Thanks for your feedback Cody! It's still a challenge for me on how can I make my code less complicated. I will surely visit these links that you provided.

  0