Code review

please review my code and tell me what i can do to improve

Do you have any formatted code to post or a link to a GitHub repo?

<!DOCTYPE html>
<html lang="en">
    <meta charset="UTF-8" />
    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    <link rel="stylesheet" href="stylesheet.css" />
    <div class="header">
      <h1>Dasmoto's Art & Crafts</h1>

    <div class="brushes">
      <img src="../HTML/assests/hacksaw.jpeg" alt="Image of Brushes" />
      <h3>Hacksaw Brushes</h3>
        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>.

    <div class="frames">
      <img src="../HTML/assests/frames.jpeg" alt="Image of Frames" />
      <h3>Art Frames (assorted)</h3>
        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>

    <div class="paint">
      <img src="../HTML/assests/finnish.jpeg" alt="Image of Paint tubes" />
      <h3>Clean Finnish Paint</h3>
        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 /
    background-image: url("../HTML/assests/pattern.jpeg");
    font-family: Helvetica;
    font-size: 60px;
    font-weight: bold;
    color: khaki;
    text-align: center;
    font-family: Helvetica;
    font-size: 32px;
    font-weight: bold;
    color: white;

.brushes h2 {
    background-color: mediumspringgreen;

.frames h2 {
    background-color: lightcoral;

.paint h2 {
    background-color: skyblue;

    font-weight: bold;
    font-family: Helvetica;
    color: blue;

Greetings bro

There are several things that you can improve in your code in my opinion :white_check_mark:

HTML part

  • Instead of using div class="header" you can use semantic HTML tags like in this case → <header><h1>Dasmoto's Art &amp; Crafts</h1></header>
  • Other small detail that you could apply in your future HTML documents it’s using HTML entities to describe symbols: &amp; = &, benefits? It’s better for screen readers, and the browser will parse the symbol better while displaying it.

CSS part

  • The header’s h1 title It’s too big, and it can be improved by showing it within 32-40 px which allows anyone to see a decent title
  • h2 elements borders can have a small padding to make them look more attractive

Hope these pieces of advice can help you to improve, enhance and better your code bro, I couldn’t watch the images, therefore i couldn’t take a full preview of your webpage