hillbillyfunk (AT) gmail (DOT) com wrote:
Quote:
[...] Please post any critiques as well as errors that cause
it to load incorrectly, display incorrectly or misspelling that you
might run across.
Viewed with Mozilla v1.7.12.
|
- HTML 4.01 Transitional is for legacy pages, not new. Use HTML 4.01 Strict.
- As noted by bts, a large number of HTML and CSS validation errors.
- The CSS validation errors prevent your layout from displaying correctly.
For instance the "Located on the ..." paragraph is not positioned in the
white area because of an error in .data_located.
- Use of tables for layout. Your site layout is fairly straightforward;
tableless layout would be simple to achieve.
- Inconsistent use of quotes around tag values. Sometimes double quotes,
sometimes single, sometimes none at all. There is no downside to quoting
all tag values.
- "tag soup" and "style swamp" for presentation. You sometimes use
style="" to replace <font> (and similar) elements. That only changes the
method without fixing the problem.
- Use of for presentation. Might as well use a "spacer" GIF. Use
CSS instead.
- You have a lot of redundant attributes in the CSS, for instance
font-size:12px. And no use whatsoever of the Cascading aspect of CSS.
- I have a minimum font size set for my browser, a size that is larger
than the small font size (12px) you have. Your site does okay with that.
Also note that IE users cannot increase the font size because you have
spec'd it in px, at a size that is 75% of standard.
- The link appearance is inconsistent. The left nav has a :hover defined.
The top nav does not.
- The left nav :hover fore-/background contrast is very poor. If anything,
the contrast should increase to heighten interest and readability.
- The link in the left nav for the current page is not disabled. Doing so
provides additional feedback to the visitor about location within the site.
- When the text wraps in the left nav, there is no obvious separation
between the links until I hover over one.
- The heading color is the same as the link color. Because of the
inconsistent link presentation, I did not know if they were links until
the mouse hovered over them.
- Non-navigation links are normally underlined. While ugly, it is what
visitors expect. Removing it makes finding links and Easter Egg hunt.
Another option is to create an obvious link style for your site.
If you change the color of one link pseudo class (:hover, :link),
change them all. (Spec them in this order: :link :visited :focus :hover
:active.)
- You use the <hx> elements for presentation, not markup. There is an <h3>
with no h1 or h2.
- The superscripted ".00" in "12.00" is very odd.
- I had to disable Javascript to stop that annoying blinking image slide
show. If you absolutely must have it, have it loop only once. Or, better,
provide a button to allow the visitor to choose to see it.
- Does N8's license require that you provide a link to their site? If not,
why is it there?
- Your images file sizes are rather large. You can save JPEGs at a lower
quality setting, say 75%, to reduce the size (for instance, from 20 KB to
5 KB). Video monitors are low resolution devices.
- Create a separate background image for the non-home pages. Clipping off
3/4 of the main image is a bad choice; the whole image must still be
downloaded. Don't waste the bandwidth or the visitor's time.
- The b&w photo of the "hanging rock" is pretty bad. Contrast is too high
and there is no spatial reference to decide what it is. (I am guessing
that is a hanging rock from the name of the campground.)
- In general your images are obviously quick snapshots. Hire a photographer.
- I never saw the stat counter because my browser only allows images from
the original site.
IMO, using an image to load a script is questionable practice at best;
it is sneaky, underhanded, and a betrayal of trust between you and the
visitor. A common use is delivering trojans and virsuses in email.
If you must have such a function, publish a site Terms of Use that
explicitly states that you may do these things.
--
jmm dash list (at) sohnen-moe (dot) com
(Remove .AXSPAMGN for email)