Hi Adam
Your code looks good. However, you could simplify it a bit.
This is a page that could use fewer divs. For instance
<div id="nav">
<ul id="navlist">
<li><a href="index.html">Home</a></li>
<li><a href="content.html">History</a></li>
<li><a href="content.html">Upcoming Events</a></li>
<li><a href="content.html">Past Productions</a></li>
<li><a href="content.html">Contact Us</a></li>
</ul>
</div>
It wouldn't take much time to remove the nav div and just associate your
styles to navlist. Right now, you are using rules such as #nav ul. Just put
the rules on #navlist.
In the news div, you could apply the styles to the news div and remove the
class="news" unless you expect to have a variety of dls in that div
#news dl {rules associated with dl.news}
#news dl dl {rules associated with dl.info}
I would change class="small" to class="fineprint" or "newsdetails". This
gives you the future flexibility to change background, colors, positions etc
without having to look for the small text.
<div id="news">
<h3>Latest News</h3>
<dl class="news">
<dt>July 1, 2005</dt>
<dd><h4>Tickets available for
Rosencrantz & Gildenstern are dead!</h4></dd>
<dd class="small">
Tickets are on sale for our
amazing performance of the hilarious Tom Stoppard play. Be sure to book
early to avoid disappointment.
<dl
class="info"><dt>Adults:</dt> <dd>$25</dd></dl>
<dl
class="info"><dt>Concession:</dt> <dd>$20 </dd></dl>
</dd>
</dl>
<dl class="news">
<dt>January 1, 2005</dt>
<dd><h4>Auditions open for Romeo
& Juliet</h4></dd>
<dd class="small">Come and audition
on thursday nights, at Blacktown RSL</dd>
</dl>
</div>
This is also excessive
<div id="feature">
<div id="feature_top">
<h3>Now Showing</h3>
</div>
<div id="feature_image">
<img src="images/cosi_icon.jpg"
alt="cosi icon" />
</div>
<div id="feature_body">
<p class="small"><strong>We have a
show that is now on at the blacktown RSL.</strong> Be sure to watch it,
there's no doubt that it will be fantastic.</p>
</div>
<div id="feature_bottom">
<p class="small">For any <a
href="home.html">questions</a>, please contact <a
href="mailto:[EMAIL PROTECTED]">[EMAIL PROTECTED]</a></p>
</div>
</div>
Instead of all these divs, try :
#feature h3 {....}
#feature p {...........}
#feature p.feature_bottom {...............}
#feature img {..........}
<div id="feature">
<h3>Now Showing
<img src="images/cosi_icon.jpg" alt="cosi icon" />
<p >
<strong>We have a show that is now on at the blacktown RSL.</strong>
Be sure to watch it, there's no doubt that it will be fantastic.</p>
<p class=" feature_bottom ">
For any <a href="home.html">questions</a>, please contact <a
href="mailto:[EMAIL PROTECTED]">[EMAIL PROTECTED]</a></p>
</div>
While this may seem like an exercise in nitpicking, imagine how much easier
it will be for the next programmer if they can just add an h3 or a paragraph
without wondering if they need to wrap it in a div and what was the name of
that div.
I think you could cut your div count in half fairly easily. Think of it as a
challenge to see how many you can remove and replace with descending styles.
Ted
www.tdrake.net
******************************************************
The discussion list for http://webstandardsgroup.org/
See http://webstandardsgroup.org/mail/guidelines.cfm
for some hints on posting to the list & getting help
******************************************************
