What tha?

Hi,

I’ll list a few for you :slight_smile:

Is it a bird heading or a plane paragraph?


<h2>
  <p>The Umbrella Arts Organization for Cass County, Minnesota</p>
</h2>

It can’t be both a heading and a paragraph. It’s either one or the other.

It is invalid to nest block level elements inside heading tags or p tags.They can only contain inline elements such as spans etc. As you aren;t even appplying styling to both elements then I don;t see why you added it anyway.

Just use this:


<h2>The Umbrella Arts Organization for Cass County, Minnesota</h2>

If you need an extra hook for styling then add a span.


<h2><span>The Umbrella Arts Organization for Cass County, Minnesota</span></h2>

Of course you can style the span however you like and make it look like a block element if needed.

You’ve wrapped a div around your ul navigation but it is a wasted element as you have no styles applied to it.


[B]<div id="navBar ul">[/B]
                <ul>
                    <li><a href="index.html">Home</a></li>
                    <li><a href="attractions.html">Coming Attractions</a></li>
                    <li><a href="photos.html">Pictures</a></li>
                    <li><a href="members.html">Member's Page</a></li>
                </ul>
            </div>


Don’t add elements unless they are really needed - remove the div and style the ul directly. (BTW what’s with the class of ul ? That’s a bit confusing and not a suitable name for a class)

As I said above p elements cannot contain block elements and you can’t do this:


<p>
        <div style='width: 120px; height: 60px; background-image: url( 
http://vortex.accuweather.com/adcbin/netweather_v2/backgrounds/spring1_120x60_bg.jpg ) 
; background-repeat: no-repeat; background-color: #607041;' >
            <div 
style='height: 45px;' >
                <script 
src='http://netweather.accuweather.com/adcbin/netweather_v2/netweatherV2.asp?partner=&tStyle=normal&logo=0&zipcode=56452&lang=eng&size=7&theme=spring1&metric=0&target=_self'></script>
            </div>
            <div 
style='text-align: center; font-family: arial, helvetica, verdana, 
sans-serif; font-size: 10px; line-height: 12px; color: FDEA11;' ><a 
style='color: #FDEA11' 
href='http://wwwa.accuweather.com/index-forecast.asp?partner=&traveler=0&zipcode=56452' 
Weather Forecast</a></div>
        </div>
        </p>


Remove the surrounding p element, you can always make space by applying margins to the div so don’t add breaks or other elements just to space things out.

For new sites use a strict doctype instead of transitional as there should be no need to support obsolete and deprecated behaviour these days.


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Untitled Document</title>
</head>

<body>
</body>
</html>


That’s a start :slight_smile:

Thank You:cool: