Tek-Tips is the largest IT community on the Internet today!

Members share and learn making Tek-Tips Forums the best source of peer-reviewed technical information on the Internet!

  • Congratulations Mike Lewis on being selected by the Tek-Tips community for having the most helpful posts in the forums last week. Way to Go!

Early site review 3

Status
Not open for further replies.

maxformed

Technical User
Apr 22, 2003
168
US
Starting a small, personal site - uploaded today and know there's some stuff that's screwy. My only previewing option are Mac IE and NN - any other view appreciated.

Oh - the contact form doesn't work yet - my host is down for another couple of hours and have to get an email alias for some pages and the cgi stuff ready.

the site:


Thanks,
Max

 
In IE6, 1024* res

Nice site!
Simple, plain, good use of colour.

One big drawback, as Wullie mentioned, is the background image. It looks good on the home page (and should scroll with the page) but it doesn't look great on HowWeDoIt.html or services.html

Get rid of the mailto tag, spammers will git ya!






- É -
 
maxformed, only one comment: use client-side validation on the 'Contact Us' page, I just sent you a blank email. This can get annoying if someone sends you a million blank emails.

I have this little thing, Advanced Delusionary Schizophrenia with Involuntary Narcissistic Rage.
It's no big deal really...
 
what he means is that check that some kind of info is sent to the page (using Javascript u can validate the page)...

Known is handfull, Unknown is worldfull
 
maxformed, Client-side validation means a validation is done on the user’s PC, making sure that the form meets all criteria’s before its been submitted.
For instance, if the name or e-mail is blank the form will not be sent. A sample of client-side validation is:
Code:
<script language=&quot;JavaScript&quot;>
function myValid() {
  //creates a var that holds the 'document.formName'
  var Gm = document.formName;
  
  //if the name field's value is blank (null)
  if(Gm.name.value == &quot;&quot;) {
    //throw an alert
    alert(&quot;Please fill in a name&quot;);
    //put focus on the field, very helpful
    Gm.name.focus();
    //return false
    return false;
  }
}
</script>

<form name=&quot;formName&quot; method=&quot;post&quot; onSubmit=&quot;return myValid();&quot;>
  <p>Name: 
    <input type=&quot;text&quot; name=&quot;name&quot;>
  </p>
  <p><input type=&quot;submit&quot; name=&quot;Submit&quot;></p>
</form>

Make sure that the formName, field name are the same. Otherwise you get an error.
You can also have server-side validation via ASP, Coldfusion, CGI, etc. But client-side is faster. And client-side throws an alert so there's no need for the user to hit the 'Back' button like they would if server-side is used.
Hope this was helpful. :)

I have this little thing, Advanced Delusionary Schizophrenia with Involuntary Narcissistic Rage.
It's no big deal really...
 
maxformed, glad I could help and if you need any more help, simply as us.
Thanks for the star. :)

I have this little thing, Advanced Delusionary Schizophrenia with Involuntary Narcissistic Rage.
It's no big deal really...
 
No problem. I love these forums - everyone is so helpful. I'm not so good with the scripting stuff for the most part, and always appreciate the constructive criticism.

Max

 
just a small addon:
instead of
if(Gm.name.value == &quot;&quot;)

use
if(Gm.name.value == 0) //every where.

the difference?
in the first case the user can get away with blank spaces. in the second they have to enter some value (ofcourse the arent allowed to enter 0)...



Known is handfull, Unknown is worldfull
 
&quot;You can also have server-side validation...&quot;

You should always validate fields coming into your server in your server-side scripts, in addition to any validation you may do at the client side.

Surfers may have Javascript switched off. Hackers might send values direct to your CGI script without actually using your form at all. You can't rely on the client exclusively to do your validation, though as GUJUmOdel says it's friendly to do it there as well.

-- Chris Hunt
 
cian,
I read your response and found something that interest me greatly
You said to get rid of the &quot;mailto tags&quot;, what exactly do you mean?
I have a few sites that have a contact page with an email address and I am getting all kinds of junk mail.
Thanks,
Dave
 
Hi Dave,

&quot;Get rid of&quot; was a bad choice or words, &quot;disguise&quot; might be better.

You're getting junk mail because certain robots have scanned your pages, found the mailto tag and stored the associated email addresses in a database to be used to send you all kinds of junk!

There's many easy javascript solutions:

A contact form is another way to go, perhaps a better way as 10% of users don't have javascript enabled.





- É -
 
great looking site, except the fuzzy headings are a little out of place/ annoying. otherwise, great colors. I love to see simple, clean and fast loading pages. It's sick how many people will fill a page up with useless java script and flash. (not that either don't have their places..)

 
Thanks, imstillatwork. I'm a little unsure of the fuzzy headings myself - initial thought was subtelty - the fuzz to make them look twice, but not overwhelm the text.

Everyone has been great. cian - i'll check those links today.

Thanks,
Max

 
maxformed, I took a look at your site and again and noticed a few things:
1) You have the W3C check icons for HTML and CSS, there really is no need for the user to see this. The W3C checks are for the designer to see and know that their coding works and is acceptable for the WWW. I would take it out.

2) I'm not too thrilled with the menu options roll-over colot schema. Looks way to &quot;pukish&quot;, know what I mean??

3) Under the CONTACT US form, I would rather have a client side validation. The reason is that in client-side the alert is thrown on the same screen as the form. The way you have it now, causes a new window to load with the alerts forcing the user to go back and fill in the fields. Unnecessary clicking. Client side valiadtion also helps cause you can send the focus to the field in question, making the user aware where they missed it. Don't get me wrong, nothing wrong with the validation the way you have it now, but you should always try to avoid unnecessary clicking.

I have this little thing, Advanced Delusionary Schizophrenia with Involuntary Narcissistic Rage.
It's no big deal really...
 
Status
Not open for further replies.

Part and Inventory Search

Sponsor

Back
Top