fix: semantic issue in 17-make-it-count

[madebysergio]

Fix: Improper use of h3 tag inside ul

• This PR fixes an improper use of an h3 tag inside a ul tag.

Changes:

• h3 tag has been wrapped inside of a li tag.
  • Note: The CSS removes the user list style added after wrapping the h3 tag in a li tag.

Related issues:

• This PR closes Improper Use of <h2> Tag Inside <ul> Element #27.

[Dusch4593]

Hey @beingsie! Thank you so much for flagging this!

You are correct about how heading elements (<h1> - <h6>) should be directly placed inside a <ul> unordered list element. Before we merge this PR, might suggest the following change:

<h2>My Feed</h2> <ul id="post-list">

Since this header is meant to represent all of the list items inside of the <ul>, I think it makes more sense to move it out of the <ul> (rather than make it a list item like you did in your change).

Let me know when you've made this change and I'll be happy to merge! 😄

[Dusch4593]

Hey @beingsie! Since we're past the cutoff time for our monthly challenge, I went ahead and made this change so we can merge this PR! 🚀