...

View Full Version : Site review, please.



student101
01-20-2009, 11:59 AM
Serious review needed.
I had a website re-developed from a co.za to a org.za site

Please give me valid and useful feedback on the org.za site, the good, the bad and the ugly.

First timer...
Run some tests and currently working on cleaning up the validation issues.
Found a couple of free sites that help with keywords, description and so on... Some of them were harsh! others very useful.

Cheers

jerry62704
01-20-2009, 06:20 PM
I like the tabs, but the highlighting of the current one should carry across the page. When I see the green, it makes the home look like it is behind the "current" page rather than the current page.

The white on light blue on the bottom is hard to read.

Fumigator
01-20-2009, 06:27 PM
Nice... good readability, looks professional. One suggestion would be on the tabs that require a log in, instead of just prompting for login info, put a little message "That feature requires that you log in. If you are not yet registered, click on "register now". It's fast free and easy!" or whatever.

When I click on a topic on the Lost and Found page, something disasterous happens to my screen-- I couldn't really tell what it was because I went into seizures at that moment, but I think the background started pulsating and wouldn't stop until I clicked anywhere on the page. Yeah... a fun effect, but really really horrible in reality. I wouldn't grey out the page either; at the very most I would just make the div's background a pale yellow (#ccccf0) which goes away when you mouse over the div or click anywhere.

Also any of the link exchange links breaks the CSS of the page.

student101
01-20-2009, 07:25 PM
When I see the green, it makes the home look like it is behind the "current" page rather than the current page.
Will get that sorted, hopefully by tommorow...

The white on light blue on the bottom is hard to read
Didn't see that one, will play with the colours a bit.

Thanks for yout input, really cool!

student101
01-20-2009, 07:31 PM
One suggestion would be on the tabs that require a log in, instead of just prompting for login info, put a little message "That feature requires that you log in. If you are not yet registered, click on "register now". It's fast free and easy!" or whatever.

Sounds cool, will try and implement it.

When I click on a topic on the Lost and Found page, something disasterous happens to my screen-- I couldn't really tell what it was because I went into seizures at that moment, but I think the background started pulsating and wouldn't stop until I clicked anywhere on the page. Yeah... a fun effect, but really really horrible in reality. I wouldn't grey out the page either; at the very most I would just make the div's background a pale yellow (#ccccf0) which goes away when you mouse over the div or click anywhere.
It's supposed to scroll to the area on the page, I see what you mean, just resized the screen to 800x600.
Thanks, will have to fix that one.

Also any of the link exchange links breaks the CSS of the page.
What do you mean here? Just need a bit more info.

Thanks for yout input, really cool!

Fumigator
01-20-2009, 07:34 PM
This page has no CSS styling:

URL removed by user request

student101
01-20-2009, 07:39 PM
Funny thing about that, you are the second person to tell me that this page looks blank.
It was suggested by my chick, that I add a tag cloud or a few categories like on the URL removed by user request page

Thanks, will do it.



What about the speed? Does it load quick enough?

Fumigator
01-20-2009, 09:47 PM
It's more than blank; it's unformatted. See attachment (assuming I can attach the image).

student101
01-20-2009, 10:00 PM
Wow that's bad?
I don't see that in the image.

It's fine here, FF and IE7, cleared TEMP and CACHE - right now.


Sure there isn't a tool or something that disables scripts or CSS on your browser?
If I disable linked stylesheets (inside web developer toolbar in FF) I get the same image as you posted.
See attached...

student101
01-21-2009, 03:21 PM
highlighting of the current one should carry across the page.
Fixed!

The white on light blue on the bottom is hard to read.
Fixed!

Hope it looks better.

What about the speed? I fear I may have to redo alot of this site, maybe even dump some neat features.

Cheers

Fumigator
01-21-2009, 05:00 PM
It might be the CSS file is blocked at my work place but I can browse directly to this file:

URL removed by user request

And I can see it just fine. So... *shrug*

abduraooft
01-21-2009, 05:26 PM
On home page it's

<link href="css/BrightSide.css" rel="stylesheet" type="text/css" />
where as on the other

<link rel="stylesheet" href="style/BrightSide.css" type="text/css" />

Are they supposed to use different stylesheets?

student101
01-21-2009, 07:17 PM
Are they supposed to use different stylesheets?
Yes each section has it's own folder... ie; own but similar stylesheet.

On a totally different point; SPEED!


I tested it with a Web Page Analyzer (http://www.websiteoptimization.com/services/analyze/) and it says:
15 HTTP Requests
I also tested it with Firebug and it says:
55 requests 247 KB (0 b from cache) 18.49s
Without the slide show script it says:
15 requests 69 KB (0 b from cache) 5.74s

I then tested a different slide show and it says:
40 requests 200 KB (0 b from cache) 6.3s //Major difference in load time.

I haven't had time to implement all the needed changes just yet, but will soon.

Apostropartheid
01-21-2009, 07:43 PM
Yes each section has it's own folder... ie; own but similar stylesheet.
Wouldn't it be easier to maintain 1 stylesheet and use body IDs?
It cuts down on HTTP requests, which on faster connections take longer than the actual download, and allows caching.

student101
01-21-2009, 08:09 PM
Wouldn't it be easier to maintain 1 stylesheet and use body IDs?
It cuts down on HTTP requests, which on faster connections take longer than the actual download, and allows caching.
I am no genius at CSS, how does that work?

Apostropartheid
01-21-2009, 09:34 PM
Something likeeee...
styles.css


html {
font: .817em/1.8 'lucida grande', 'lucida sans unicode', sans-serif;
}
/* rest of base stylesheet */
/* makes home.html's body black */
body#home {
background: #000;
}
/* makes single pages' body white */
body.single {
background: #fff;
}


home.html


<!doctype html>
<html lang="en">
<head>
<title></title>
<link rel="stylesheet" type="text/css"
href="styles.css">
</head>
<body id="home">
<p>Home.</p>
</body>
</html>
single.html


<!doctype html>
<html lang="en">
<head>
<title></title>
<link rel="stylesheet" type="text/css"
href="styles.css">
</head>
<body class="single">
<p>Single page.</p>
</body>
</html>
This can also be used for tab navigation as a positive side-effect.

student101
01-22-2009, 12:01 AM
I am trying to understand, why your method makes it faster OR easier ?

Currently a similar style sheet with a few changes is being used in a different folder.

If I use your method the main CSS file wont be 8KB
I may need three or four files to do the same thing, which may create more HTTP requests and or objects on the page.

Would it be better then, to have each page set with it's own syle sheet? OR have the style embedded in each page?

Your method brings forward many questions for me :confused:.
(I prefer one style sheet, but isn't there a size and a standard?)

Apostropartheid
01-22-2009, 12:04 AM
What? I thought you said each page had its own independant stylesheet! My method would only include one stylesheet.

student101
01-22-2009, 08:56 PM
Yes it's one style sheet but in 3 different folders... Message board, Pet classifieds and a Link exchange folder
I see what you mean, will have to play around some more with that one.
I have changed the entire site to include a new slide-show! It's faster and unbelievable!
I fear I may have made it worse than before;
Firebug says: 47 requests 219 KB (0 b from cache) 10.96s
Website optimizer says: Warning! The total number of objects on this page is 34 which by their number will dominate web page delay
This makes no sense, because it is so much faster!


Thank you all for your valid input and help here.

Cheers

The_Return
01-25-2009, 04:57 AM
Its a good color sceme but should expand it a little.

student101
01-25-2009, 07:52 AM
Its a good color sceme but should expand it a little.
What do you mean by expand it?

Cheers



EZ Archive Ads Plugin for vBulletin Copyright 2006 Computer Help Forum