Code Review: Dasmoto's Arts & Crafts project


<html lang="en">

    <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">

                <!-- Header -->

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

                <!-- Brush Section -->

        <div class="brush">
            <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 /

                <!-- Frames Section -->

        <div class="frame">
              <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 /

                <!-- Paint Section -->

        <div class="paint">
              <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>

And my CSS

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;

p {
    color: black;
    text-align: left;

.mainHeader {
    text-align: center;
    background-image: url(;
    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;

.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.