Honest opinion

[mitch2801]

Site Name: McDonalds Wedding Cars
Site URL: www.mcdonaldsweddingcars.co.uk
Site Descriptuion: Wedding car hire company in Greenock
My Comments: I built this site as a first attempt for a mates business. he seems happy enough but I would like to know anything that you guys think could improve it. Also I find that when I have clicked on a navigation button and use the back button on the browser it does not go back a page. And would also like to know how I can get the subject "Wedding Car Query" to automatically appear in the subject box on clicking email. Any help and honest opinions good or bad would be appreciated. thanks

[nickstan]

Hi m8,

My back button works fine all through the site?
For the email subject just alter your email link to like the one below:

mailto:mcdonaldsweddingcars@yahoo.co.uk?subject=Wedding Car Query

As for the site for a few pointers I'd align the outer graphics right up to the top and side, and change the colour of the links from the green when they've been clicked to something that stands out a bit more.

It's also a bit graphic heavy. I'd use normal text for the writing thoughout the site rather than images. With the droip shadow some pages are also a bit hard to read.
Also you've stretched some of the images such as the car on the front page so it pixallates a bit.

Anyway's I'd recommend you surf around some of the many other web design companies sites and steal ideas.

Not a bad first attempt though

Cheers nick

[andythetwig]

tip: don't use a real email on web pages unless you want to get added to every mailing list under the sun by the malicious spambot from hell.

Instead use:

<script language=javascript>emailE=('yourname@' + 'yourdomain.com'); document.write('<A href="mailto:' + emailE + '">' + emailE + '</a>') //--></script>

that way you're invisible

[add] flipping forum is parsing my javascript into invisibility. arse [/add]

[designplace]

Do a find and replace on all the <'s and change them to &amp;lt; - change the >'s to &amp;gt; - then post the code.

Should appear fine.

Test:
&lt;script language="JavaScript"&gt;

[Marcus Berry]

On the Gallery's page you have the link to return to the 'Home Page', so if I want to look at another picture I'd have to go back into gallery again and wait for the pics to load.

Perhaps just have a back button alongside each photo that returns you to the Gallery page.

At the bottom of the links page you seem to have shortcut text links to part of your site. Never seen this before and would say it detracts from the general layout of your links page which has banners for other sites you link too.

Apart from that, it doesn't look too bad and has potential...

Well done...

[brumbino]

The cols are a bit bright...

And where's the Bigmac vouchers?

[Poobar]

K, first off the email thingy you wanted the links like this:

<a href="mailto:yourdomain@yourdomain.com?subject=This Is The Subject Line Of My Email">yourdomain@yourdomain.com</a>

when you click the link it should pre-fill the subject box.

Secondly I think everyone above has given good advice so i'll just repeat a couple things I thought you should change. Lose the shadow on the main body text and change it to normal html based text rather than an image or series of images.

You can also do the date thing without using a form field, just plain text too, just stick this code in where you want it on the page:

<code><script>

document.write("Today is " +
	dayName[now.getDay()] + ", " +
	monName[now.getMonth()] + " " +
	now.getDate() + ".")

</script></code>

OK, for some stoopid reason the code wont display so go to this url and take yer pick, http://javascriptkit.com/script/cutindex1.shtml i'd suggest going for one of the simpler and straight forward ones because the fancy ones are a bit too gimmicky!

I dont think its a bad first stab at all!

Hope it helps!

[mitch2801]

Thanks lads for all your input. You have given me a couple of good tips which I'll try out.

Thanks again