...

View Full Version : My first site.



GraphicGoodies
07-04-2008, 08:52 AM
http://www.graphic-goodies.com/

This is my first website and any suggestions to improve it are appreciated. :)

_Aerospace_Eng_
07-04-2008, 11:27 AM
In Firefox 3 your content is below your navigation rather than next to it. Funny how you say this

Best viewed in Firefox 3, Internet Explorer 7, and Opera 9 browsers in a maximized window.

I'm using a 1280x800 resolution at full screen.

I don't like how the text moves on your navigation when you mouseover the links.

Why do you have two navigations?

And your code doesn't validate. See your errors here (http://validator.w3.org/check?uri=http%3A%2F%2Fwww.graphic-goodies.com%2F&charset=(detect+automatically)&doctype=Inline&group=0).

fl00d
07-04-2008, 03:26 PM
I use a 1280x1024 resolution and there is a massive purple space between the header and the content, both don't actually fit on my screen at the same time - I have to scroll down quite a bit to see content.

BWiz
07-04-2008, 03:41 PM
It's pretty slick for a first page. My didn't look half as good as that. But then again, my first web page's layout didn't break. Was there supposed to be a huge gap between the header image and the actual content. It seems that your first navigation, to the right, prevents the actual content from following the header image. I couldn't take an image of the break, because it is so huge, but here: an image of the gap (http://i316.photobucket.com/albums/mm347/bwiz91/graphic-goodies-gap.png).

Your color scheme leaves a little to be desired. Sure, using shades of one color can do the job, but in your case, you have a lot of content, and all of it following this burgandy and white (plus a little green) color scheme makes the site a little visually boring. May I suggest visitng ColorSchemer (http://www.colorschemer.com/schemes/) or ColorLovers (http://www.colourlovers.com/)? Both are great tools for web designers when choosing color schemes. ColorLovers is better, as you don't have to purcahse any software to use it, and download options include the HEX value (as far as I can remember).

You have far too many advertisments in your page. They bog it down, and are complete eyesores to the rest of your site. Sure, you want to make money of your site, but you have so many adverstisments that it detracts from thr purpose of an advertisment.

I would remove the entire links section and place it on a seperate page. At first glance, they looked like mini advertisments. Still, their flashiness is slightly nauseating.

GraphicGoodies
07-05-2008, 02:04 AM
Thanks for the reviews...

I added some things to my CSS which should fix the problems with other resolutions. How does it look now? I also moved the Links to a new page.

ArcticFox
07-05-2008, 08:16 PM
And your code doesn't validate. See your errors here (http://validator.w3.org/check?uri=http%3A%2F%2Fwww.graphic-goodies.com%2F&charset=(detect+automatically)&doctype=Inline&group=0).

Did you actually look at what the validation errors showed?!




Error Line 9, Column 84: document type does not allow element "LINK" here.
Line 10, Column 76: document type does not allow element "LINK" here.
Aren't links suppose to be between <head></head>?



Line 37, Column 6: end tag for element "HEAD" which is not open.
Interesting, because when I view source, I CLEARLY see an opening <head> tag at the top of the page.



Line 38, Column 5: document type does not allow element "BODY" here.
Oh, that's right, the body goes here... </html><body>



Line 124, Column 65: cannot generate system identifier for general entity "zs".
Line 124, Column 65: general entity "zs" not defined and no default entity.
Line 124, Column 67: reference to entity "zs" for which no system identifier could be generated.
"[...]caused by unencoded ampersands" - riiiight. It's a frigg'n src link, leave it alone!


Validation is for chumps who can't think for themselves. :thumbsup:

_Aerospace_Eng_
07-05-2008, 08:23 PM
@ArticFox Yes I looked at the errors. That first one is because of this.

<meta name="verify-v1" content="VopB68SBL9/ssg6ietju8lvu3fUV4USs/WsadAYGBcc=" />

They are using the wrong syntax to close a meta tag. They aren't using XHTML so no need for the /> part.

The rest can be solved by simply using &amp; instead of &.

ArcticFox
07-05-2008, 08:52 PM
@AE: Do you know what that code is for?


<meta name="verify-v1" content="VopB68SBL9/ssg6ietju8lvu3fUV4USs/WsadAYGBcc=" />

_Aerospace_Eng_
07-05-2008, 08:57 PM
Yeah I do. Its for google analytics. Google is just verifying the name and content. It doesn't need the /> part.

ArcticFox
07-05-2008, 09:02 PM
Does that /> being in the code affect the look, feel, accessibility in any browser out there?

No.

But according to the "validator" it's an error/problem and needs to be corrected or you won't get your W3 graphic cookie. Why?

_Aerospace_Eng_
07-05-2008, 09:09 PM
Because its XHTML self closing syntax. They OP isn't using XHTML. They are using HTML.

ArcticFox
07-05-2008, 09:13 PM
Is that really a problem for visitors? That /> would keep pages from loading, colours from showing correctly, positioning issues or text readers from pronouncing things...?

I'm not understanding what the actual problem is.

_Aerospace_Eng_
07-05-2008, 09:22 PM
He asked for a review and from a review in a "coding" forum you usually get your coding errors pointed out to you...

ArcticFox
07-05-2008, 09:33 PM
Ah yes, the left turn at Albuquerque.

Got it.

oracleguy
07-05-2008, 10:23 PM
He asked for a review and from a review in a "coding" forum you usually get your coding errors pointed out to you...
Agreed.

@ArcticFox, Please stop trying to create a flame war. Asking about the validation errors is one thing, discussing the importance and relevance of valid code is not appropriate for a site review.

ArcticFox
07-05-2008, 10:28 PM
Uh, huh.

Apostropartheid
07-06-2008, 12:47 AM
Best viewed in Firefox 3, Internet Explorer 7, and Opera 9 browsers.
It's a classic sign of a newcomer to tell people to view your website in a certain way.
There's a slight freedom which comes with a medium that can be viewed by anyone, anywhere. You shouldn't tell people to change their habits for you. It's just odd. Traffic colours don't suddenly invert at a specific junction, unless you've ignored the traffic lights, sailed on right by and suffered a severe accident. But anyway, drop the message. It can even be taken offensively, you know.

Onto business, although I like Tahoma, you've not specified any backup typefaces. Not all operating systems will have Tahoma installed (no Linux distributions I know of come with it preinstalled and Mac OS definitely doesn't.) You can specify a comma-separated series in the font property.


element {
font-family: tahoma, verdana, sans-serif;
}
See HTML Dog CSS reference (http://www.htmldog.com/reference/cssproperties/font-family/).

And the colours of the links on the sidebar are too dark. I can barely make them out, and on a less-than-well-calibrated-screen it might be illegible. Lighten them up to increase readability.

BWiz
07-06-2008, 06:24 AM
It looks much better now that the content has fallen all toegther. I still think that you should vary the color scheme, but thats up to you.

Right now, your major issue is the banner advert. Because of your monochromatic web site, the first thing the viewer will see is the ad. The brugandy colors around it acts like a border, and a human's eye automatically focuses on whats in the border. It is as if your web site is selling the advertisment. Is that what you really want when the user opens up their web site? Banner adverts are notoriously eye catching. There is a reason why they are like this - because they are usually put off to the side, and they need to scream for attention. Your layout does the screaming for the adverts right now.

You also have FOUR Google AdSense text links. In my opinoin, if the user doesn't click on one, they will click on none. Don't waste content space. Put up one group of text based links, or two if they are slighty discreat.

I'm also not to fond of your header title. I think you should lessen the height a little bit, and vary the colors a little.

BTW, ArticFox, do you just like hate life? You sound real vain.

ArcticFox
07-06-2008, 06:32 AM
No, I'm just tired of hearing about "validation" issues and DIVs versus TABLES. Every time I come here it's the same BS. :D

Which reminds me, when ever I ask for opinions on just how me site looks, I have to tell people to ignore the code - yet every time I'll have someone reply or PM me saying it doesn't validate... blah blah.

GraphicGoodies
07-06-2008, 11:52 AM
Thanks CyanLight and BWiz.

BWiz: Colors have always been a weakpoint for me. :\ I'm not 100&#37; fond of them either. The overall color scheme is what I want to go with, but I'm having real trouble sprucing them up and making them look better (I've tried a bunch of different things, but I'm just not very good). Any recommendations?

Sorry, but could you clarify on the banner advert? Are you talking about the top banner or the image Adsense ad? And I'm not sure what you mean by screaming adverts.

BWiz
07-06-2008, 05:02 PM
Sorry, but could you clarify on the banner advert? Are you talking about the top banner or the image Adsense ad?
The banner advert at the top, right next to the large Google AdSense links.
Screaming adverts was just my metaphor in saying that your banner advertisment detracts the user form the actual content of the web site, since it bright and colorful, and your site is monochromatic.


The overall color scheme is what I want to go with, but I'm having real trouble sprucing them up and making them look better (I've tried a bunch of different things, but I'm just not very good). Any recommendations?

I told you in my first post to head over to ColorLovers (http://colorlovers.com) or ColorSchemer (http://colorschemer.com) and choose a pallette which you think looks best. DOn't get me wrong, I like the burgandy color that you have chosen, but it is so repetitive.


EDIT:
Here is one I thought you may like: Vintage Wallpaper (http://www.colourlovers.com/palette/447936/vintage_wallpaper). You need to create an account (free) with ColorLovers before you can dowload the pallettes. Of course, you could just take a screenshot, import it into image editing software like PhotoShop and use the eye dropper tool.

GraphicGoodies
07-11-2008, 03:03 AM
I've changed the site layout, and changed a few other things as well.

BWiz
07-11-2008, 04:53 AM
Much better now. I like it a lot more. Two things, the updates panel at the top, the text is not actually in it (FF3). Also, the navigation font size (I should have mentioned this earlier) is a little too small, increase it by 1 or 2 px.

GraphicGoodies
07-11-2008, 06:28 AM
Thanks. :) Both things have been fixed.

binaryWeapon
07-12-2008, 01:02 AM
Design:
- IMO the updates box is badly place, just kind of hanging at the top of the page. I would suggest putting it in one of the content bars.
- I think you designed the site well for the intended audience.

Technical:
- Good coding styles
- No validation errors

Nice site!



EZ Archive Ads Plugin for vBulletin Copyright 2006 Computer Help Forum