clementgeorges / prj-rev-bwfs-dasmoto

Dasmoto
0 stars 0 forks source link

Div usage #2

Open farishkash opened 6 years ago

farishkash commented 6 years ago
<div class="frames"><h2>Frames</h2></div>
    <div>
      <img src="../DasmotosArtsAndCrafts/resources/frames.jpeg" alt="Art Frames"><br />
      <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. <span class="price">Starting at $2.00 / frame.</span></p>
    </div>

You can use the same div for all the content instead of two separate divs. Then you move the frames class to target the h2 instead of the div. This gives you the same effect while putting each section together in a container. This gives your site the flexibility to be manage by divs if you make future additions without changing the look over anything.

Your html refactored after this change.

 <div>
<h2 class="frames">Frames</h2>
      <img src="./resources/frames.jpeg" alt="Art Frames"><br />
      <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. <span class="price">Starting at $2.00 / frame.</span></p>
    </div>
clementgeorges commented 6 years ago

Thanks and noted, I did not notice that I did that. But will be more insightful with this information.