First Full-Stack Skill Path Project

Hello,
I just started the Full-Stack career path and just finished my first project. I was hoping to get some feedback on my code for this project. Thank you so much in advance.

<!DOCTYPE html>
<html>
  <head>
    <title>Everyday with Isa</title>
    <body>
      <a href="#contact">
      <img
       src="https://content.codecademy.com/courses/learn-html/elements-and-structure/profile.jpg"
      /> 
      </a>
      <h3>"by Isabelle Rodriguez | 1 day ago"
      <h1>"An Insider's Guide to NYFW"</h1>
      <img
        src="https://content.codecademy.com/courses/learn-html/elements-and-structure/image-one.jpeg"
      />
      <p>
        <a href="https://en.wikipedia.org/wiki/New_York_Fashion_Week" target="_blank">NYFW</a> can be both amazingly fun [&] incredibly overwhelming, especially
        if you’ve never been. Luckily, I’m here to give you an insider’s guide
        and make your first show a pleasurable experience. By taking my tips and
        tricks, and following your gut, you’ll have an unforgettable experience!
      </p>
      <h2>"Getting Tickets [&] Picking the Show"</h2>
      <img
        src="https://content.codecademy.com/courses/learn-html/elements-and-structure/image-two.jpeg"
      />
      <p>
        If you’re lucky or connected you can get an invite, sans the price tag.
        But I wasn’t so lucky or connected my first 2 years so I’m here to help
        you out. First, plan out which shows are most important to you and make
        a schedule and this is a biggie: SET A BUDGET. If you’re worrying about
        blowing your cash the whole time you won’t have fun. Then check out
        prices, days, and times and prioritize the designers you want to see
        most. Lastly, purchase your tickets and get excited!
      </p>
      <h2>"Dressing for the Shows"</h2>
      <img
        src="https://content.codecademy.com/courses/learn-html/elements-and-structure/image-three.jpeg"
      />
      <p>
        Always be true to your own sense of style, if you don’t you’ll be
        uncomfortable the whole time and it will show. Remember, NYFW is about
        expressing yourself and taking in what the designers have chosen to
        express through their new lines. Also it’s important to wear shoes
        you’ll be comfortable in all day. Obviously you want to look good, but
        you’ll be on your feet all day long, so be prepared.
      </p>
      <h4>"Related Content"</h4>
        <ul>
          <li>"How To Style Boyfriend Jeans"</li>
          <li>"When Print Is Too Much"</li>
          <li>"The Overalls Trend"</li>
          <li>"Fall's It Color: Blush"</li>
        </ul>
      <div id='contact'>
        <p><strong>email:</strong> [email protected] | <strong>phone:</strong> 917-555-1098 | <strong>address:</strong> 371 284th St, New York, NY, 10001</p>
      </div>
    </body>
  </head>
</html>

This looks good for a first project; well done! You’ve organised the code well, and left appropriate whitespace, so it’s easy to see what’s going on, which is great! One suggestion I would have is to divide it into a few sections using <div> (or some semantic elements), because it’s a good habit to get into for when you learn about styling with CSS, and if you want to give your site functionality later on (but it’s not necessary here—just good practise :slightly_smiling_face:).

Lastly, it seems like you’re missing a closing </h3> tag here:

Great work!

1 Like