3 replies [Last post]
Last seen: 17 years 17 weeks ago
Joined: 2005-06-01
Posts: 13
Points: 0

Hello, was wondering if you guys could check out a site we just finished before it goes live. Its not all css layout, the main layout is with tables, but there is a lot.



fatfreddy's picture
Deep down under
Last seen: 15 years 10 weeks ago
Deep down under
Timezone: GMT+8
Joined: 2005-03-25
Posts: 390
Points: 0

site redesign

rbones 151
why have you used tables, it's a nice enough layout and design but you could present it all with CSS? Not bagging you/it just thinking out loud.

Also just check your code

and your CSS

It would be worth you spending a bit of time optimising those images the page takes nearly 2 minutes to download on a 56k modem

Warning! The total size of this page is 544568 bytes, which will load in 108.93 seconds on a 56Kbps modem

check out

That said, i will repeat that it quite a nice looking site


The only reason for time is so that everything doesn't happen at once
--Albert Einstein

Accessible webdesign Australia - webechodesigns

gleddy's picture
sydney, australia
Last seen: 14 years 21 weeks ago
sydney, australia
Timezone: GMT+10
Joined: 2004-09-21
Posts: 596
Points: 0

site redesign

breaks down quite well with your use of lists, except for the main image and the quick links which go a bit berserk and float to the right of the page.

Would look quite nice if you could fix that up.

Last seen: 17 years 17 weeks ago
Joined: 2005-06-01
Posts: 13
Points: 0

site redesign

Thanks for the tips, I will work on the images next to get them down. All the html errors on the validation deal with two things, I am using javascript to center the page as margin: 0 auto was giving us a weird pixel bug when you hovered over links, a line would appear to the left of the link. Once auto was out, the problem disappeared. So I am using the following JS for that

<script type="text/javascript">
var margin = screen.width;
margin = (margin-780)/2 - 15;
document.write('<style type="text/css" media="screen">#container {width:780px; border: 1px solid #D1CECE; margin: 0 0 0 ' + margin + 'px; padding: 1px;}');
document.write('#subcontainer {width:780px; margin: 0 0 0 ' + margin + 'px; padding: 1px; background: #FFF;} </style>');

So thats a couple errors, the other errors deal with the flash object in the pages, it doesn't like any of the attributes I am using:

<object classid="clsid:D27CDB6E-AE6D-11cf-96B8-444553540000"
				id="intro" width="240" height="250" >
				<param name="movie" value="movie<%= choice %>.swf" />
				<param name="bgcolor" value="#FFFFFF" />
				<param name="quality" value="high" />
				<param name="wmode" value="opaque" />
				<param name="alt" value="view the video tour, click on the lower left corner of the flash animation" />
				<param name="allowscriptaccess" value="samedomain" />
				<embed type="application/x-shockwave-flash"
				width="240" height="250"
				wmode="opaque" style="border: 1px solid red;"
				name="intro" src="movie<%= choice %>.swf"
				bgcolor="#FFFFFF" quality="high" alt="view the video tour, click on the lower left corner of the flash animation"
				swLiveConnect="true" allowScriptAccess="samedomain"

This is my first attempt at using css for layouts and some of the things we were trying to do weren't working out for us so we had to use tables for some of the layout. I think is was because we use include files, can't remember though it was so long ago.

I'll take a look at the site with out css and see what the main images do.

Anything about the sub pages?

Thanks for all the feedback.