Click to See Complete Forum and Search --> : Bethel Ebensburg Review


hoffmandirt
01-09-2008, 01:12 PM
Hey guys & girls,

Would you all mind taking a minute or two to review the following site: http://www.bethelebensburg.org. I developed this free of charge for a local church and I am more of a developer than a designer so design tips are highly encouraged. I appreciate everyone's input.

Thanks!

matt.ritter
01-09-2008, 02:04 PM
The flickering nav is a bit much for me. If you wanted to keep an animation for hover on the nav I would have gone with something smoother. That is just my personal humble opinion though.

dtm32236
01-09-2008, 03:51 PM
i think the site's really nice

your doctype is incomplete, it should be:
<!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">

and you never closed the head:
<head>
<title>Bethel Assembly of God, Ebensburg, PA</title>
<meta name="description" content="Bethel Assembly of God is a thriving church in Ebensburg PA." />
<meta name="keywords" content="christian, jesus, god, bethel assembly of god, ebensburg, pennsylvania" />
<meta name="robots" content="index,follow" />
<meta http-equiv="content-type" content="text/html; charset=UTF-8" />
<link type="text/css" rel="stylesheet" href="http://www.bethelebensburg.org/css/layout.css" />
<link type="text/css" rel="stylesheet" href="http://www.bethelebensburg.org/css/tagStyles.css" />

<link type="text/css" rel="stylesheet" href="http://www.bethelebensburg.org/css/forms.css" />
<script type="text/javascript" src="http://www.bethelebensburg.org/js/swfobject.js"></script>
</head>
<body>

...just a couple errors to fix up.

good job with the site.

hoffmandirt
01-09-2008, 07:10 PM
Good eye on the head element. I can't believe I missed that. Have either of you ran the page through the w3 org validator? Can someone explain to me why the following is not liked by the validator?


# Error Line 32, Column 140: document type does not allow element "input" here; missing one of "ins", "del", "h1", "h2", "h3", "h4", "h5", "h6", "p", "div", "address", "fieldset" start-tag.

…ld" style="width:150px;height:16px" />&nbsp;<input type="submit" value="Searc

The mentioned element is not allowed to appear in the context in which you've placed it; the other mentioned elements are the only ones that are both allowed there and can contain the element mentioned. This might mean that you need a containing element, or possibly that you've forgotten to close a previous element.

One possible cause for this message is that you have attempted to put a block-level element (such as "<p>" or "<table>") inside an inline element (such as "<a>", "<span>", or "<font>").



9 more of these...

Jick
01-09-2008, 10:14 PM
It's because you need to wrap the elements in the form in another tag. A block tag like <p> or something. You could also just wrap them all in a <fieldset>. Hope that helps.