Site Review Needed - Tech Support Forums

Blazer06 · December 27th, 2004, 02:19 PM

How does this site look:

http://www.atholdailynews.com/700Design/index.php

Any compatibility errors that arise? (Tested on IE, IE Mac, Netscape Mac, Opera, FireFox)

Any suggestions?

Blaze

njolakoski · December 27th, 2004, 02:27 PM

Looks nice here, easy to navigate

cryptoguy · December 27th, 2004, 02:56 PM

I like the design. No errors that I can find.

James T · December 27th, 2004, 03:16 PM

Looks good, I'll do my usual and suggest a run through http://validator.w3.org/check?uri=ht...gn%2Findex.php
which shows a multitude of really minor errors and a couple re table layout and tr use.

edit: navigation is nice but some indication in the nav menu as to which page you are on would be good.

Blazer06 · December 27th, 2004, 04:22 PM

Indication added, per request. (orgionally had it planned in pure HTML prototype, but never had implemented it)

Some of those errors have been fixed... but I add white space to my code to make it, for example, look like this:

Code:

<html xmlns="http://www.w3.org/1999/xhtml">
  <head>
    <title>Athol Daily News</title>
    <link href="styles/styles.css" rel="stylesheet" type="text/css" />
  </head>

  <body>

<!-- Checks to see if the user is logged in -->
    <?
      if (isset($_COOKIE["atholdailynews"])) {

instead of this:

Code:

<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>Athol Daily News</title>
<link href="styles/styles.css" rel="stylesheet" type="text/css" />
</head>

<body>

<!-- Checks to see if the user is logged in -->
<?
if (isset($_COOKIE["atholdailynews"])) {

but it picks up on that. (and reports it as an error!)

Blaze

James T · December 27th, 2004, 04:43 PM

Nice.

That's very odd with the spaces. I use spacing on my own site with no validation problems. This tends to suggest that there is something else going on as well. Personally I'd hang on to the indentation irrespective.

There's another useful little package called html_tidy. Takes a page on stdin and passes out a tidied version on stdout. http://tidy.sourceforge.net/

edit: changing <br/> for <br> would fix a few as well.

Blazer06 · December 27th, 2004, 04:53 PM

Yup, plan on that. However, alot of the problem I see are becuase of the PHP generation. (seems like that anyways....)

Blaze