Responsive site with just HTML and CSS

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  Hey Bethany, great work on this challenge. I'm glad to hear you enjoyed it. Your solution looks really good and it's great that you were getting inspiration from other people's solutions to help with certain areas.

  Here are a couple of small pointers after taking a look at your code:

  • For the circle in the navigation, if you're going to keep it in the HTML like you've done I'd recommend adding the aria-hidden="true" attribute to make sure it's not read by screen reader software. You could also use a pseudo-element, like the ::before or ::after element to create this effect purely in CSS.
  • You've got a h1 and h3 headings, but no h2. Be sure to avoid skipping heading levels unnecessarily, as this can cause accessibility issues with the content hierarchy.

  I hope that helps. Keep up the great work!

  1

  ---

  

  Bethany•855

  @whimsicurl-creationsPosted almost 5 years ago

  Great tips, Matt! Thanks for taking a look at my solution and looking over the code. I will make those changes and keep that in mind for the future as well. These challenges have been great to get more experience and learn with this encouraging community.

  0

  

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  @whimsicurl-creations that's great to hear! Your solutions and the feedback you're giving is awesome. Doing the challenges and then also reviewing other people's code is an amazing way to level-up your skills. You get such a good insight into how other people approach the same problems, which is a goldmine of information!

  1