Code review

Just finsished my first project on my own and need someone to review my code and css also my finishing project with some tip and correction would be very helpful

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Arts and craft</title>
    <link rel="stylesheet" href="C:\Users\popco\OneDrive\Desktop\Coding\2ns project.html\resources\index.css">
</head>
<body>
    <H1>Dasmoto's Arts & Crafts</H1>
      <h2 class="brushes">Brushes</h2>
      <img src="https://content.codecademy.com/courses/freelance-1/unit-2/hacksaw.jpeg?_gl=1*1qsuqze*_ga*NDE1MjA5Mzc3My4xNjU3NTc4Njkz*_ga_3LRZM6TM9L*MTY1Nzk5NDMzOC4xOS4xLjE2NTgwMDQ1MDIuNTk." alt="">
        <h3>Hacksaw Brushes</h3>
          <p>Made of the highest quality oak, Hacksaw brushes are known for their weight and ability to hold paint in large amounts. Available in different sizes. <strong>Starting at $3.00 / brush.</strong></p>
      <h2 class="frames">Frames</h2>
      <img src="https://content.codecademy.com/courses/freelance-1/unit-2/frames.jpeg?_gl=1*1l8kh7j*_ga*NDE1MjA5Mzc3My4xNjU3NTc4Njkz*_ga_3LRZM6TM9L*MTY1Nzk5NDMzOC4xOS4xLjE2NTgwMDQ1MDIuNTk." alt="">
        <h3>Art Frames (assorted)</h3>
           <p>Assorted frames made of different material, including MDF, birchwood, and PDE. Select frames can be sanded and painted according to your needs. <strong>Starting at $2.00 / frame.</strong></p>
      <h2 class="paint">Paint</h2>
      <img src="https://content.codecademy.com/courses/freelance-1/unit-2/finnish.jpeg?_gl=1*1l8kh7j*_ga*NDE1MjA5Mzc3My4xNjU3NTc4Njkz*_ga_3LRZM6TM9L*MTY1Nzk5NDMzOC4xOS4xLjE2NTgwMDQ1MDIuNTk." alt="">
          <h3>Clean Finnish Paint</h3>
             <p>Imported paint from Finland. Over 256 colors available in-store, varying in quantity (1 oz. to 8 oz.). Clean Finnish paint microbinds to canvas, increasing the finish and longevity of any artwork.<strong> Starting at $5.00 / tube.</strong></p>
</body>
</html>
h1{
font-family: Helvetica;
font-size: 100px;
font-weight: bold;
color: khaki;
text-align: center;
background-image: url(https://content.codecademy.com/courses/freelance-1/unit-2/pattern.jpeg?_gl=1*1qsuqze*_ga*NDE1MjA5Mzc3My4xNjU3NTc4Njkz*_ga_3LRZM6TM9L*MTY1Nzk5NDMzOC4xOS4xLjE2NTgwMDQ1MDIuNTk.);
}
h2{
font-family: Helvetica;
font-size: 32px;
font-weight: bold;
color: white;
}
h3{
    font-family: Helvetica;
}
.brushes{
background-color: mediumspringgreen;
}
.frames{
    background-color: coral;
}
.paint{
    background-color: skyblue;
}
strong{
    color: blue;
    font-family: Helvetica;
font-weight: bold;
}
p{
    font-family: Helvetica;
}

Hi there,

So, the only thing I really noticed off-hand was this…

<strong> already makes the text it wraps bold, so you wouldn’t need to apply font-weight (unless you specifically wanted the font-weight to be even bolder)

1 Like