Code Review: Dasmoto's Arts & Crafts project

My HTML

<!DOCTYPE HTML>
<html lang="en">

<head>
    <meta charset="UTF-8">
    <meta name="viewport"
          content="width=device-width, user-scalable=no, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0">
    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Dasmoto's Arts & Crafts</title>
    <link href="style.css" rel="stylesheet" type="text/css">
</head>

<body>
                <!-- Header -->

    <header class="mainHeader">
        <h1>Dasmoto's Arts & Crafts</h1>
    </header>

                <!-- Brush Section -->

        <div class="brush">
            <h2>Brushes</h2>
            <img src="imgs/hacksaw.webp" alt="none">
                <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>
        </div>

                <!-- Frames Section -->

        <div class="frame">
             <h2>Frames</h2>
              <img src="imgs/frames.webp" alt="none">
                 <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>
        </div>

                <!-- Paint Section -->

        <div class="paint">
              <h2>Paint</h2>
              <img src="imgs/finnish.jpeg">
                 <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>
        </div>
</body>
</html>

And my CSS

body,
p {
    font-family: helvetica, sans-serif;
}

h2 {
    font-size: 32px;
    font-weight: bold;
    color: white;
    border-radius: 15px;
    text-align: center;
    width: 50%;
    margin-left: 25%;
    padding: 5px 0;
}

h3,
p {
    color: black;
    text-align: left;
}

.mainHeader {
    text-align: center;
    background-image: url(https://content.codecademy.com/courses/freelance-1/unit-2/pattern.jpeg);
    box-shadow: inset 0 -20px 8px 0 rgba(255, 255, 255, 0.2);
    width: 100%;
    height: 150px;
    z-index: 1;
    padding-top: 20px;
    position: sticky;
}

header h1 {
    font-size: 100px;
    font-weight: bold;
    color: khaki;
    margin-top: inherit;
}

strong {
    color: blue;
}

.brush h2 {
    background-color: MediumSpringGreen;
}

.frame h2 {
    background-color: LightCoral;
}

.paint h2 {
    background-color: SkyBlue;
}

.brush,
.frame,
.paint {
    margin: 150px 25px;
}

I would appreciate some feedback on this. I think I “accomplished” the task, but I still don’t like how it looks. What would you have done differently?

This topic was automatically closed 41 days after the last reply. New replies are no longer allowed.