Web Design Forum  
 
Go Back   Web Design Forum > Web and Graphic Design > Website Reviews

Like Tree1Likes

Reply
 
LinkBack Thread Tools Display Modes
Old 07-26-2011, 07:09 PM   #1
New Member
 
Join Date: Jul 2011
Posts: 9
Exclamation First CSS Site -- Please Critique

Hello all.

I just finished laying out my index page and working out most of the bugs(I think).

I'm pretty satisfied with the layout itself, but it has no color. I have no idea what colors to pick or where to put them.

I'm a much better code monkey than a designer, so please help me!

Here it is:
Fire-Ex Website

Thanks in advance!

Edit: Also, please let me know if the css doesn't agree with your browser! It appears to work fine in IE7+, Firefox 5, and in Chrome

Anyone? Bueller?

Last edited by Reed92; 07-26-2011 at 08:42 PM.
Reed92 is offline   Reply With Quote


Old 07-27-2011, 06:26 AM   #2
Platinum Member
 
Join Date: Sep 2010
Posts: 509
Default

put logo on a black/dark background .

take colours from the logo and match with blacks and silvers. the yellow for nav is a good suggestion

EDIT: is the logo yours? the edges aren't antialiased properly

Last edited by d a v e; 07-27-2011 at 08:24 AM.
d a v e is offline   Reply With Quote
Old 07-27-2011, 08:08 AM   #3
Bronze Member
 
Join Date: May 2009
Posts: 49
Default

It doesn't work in Chrome for Mac.

reset your padding to the #nav and get rid of the negative margin.
Define overlow:hidden; on the nav and it will work just a little bit better.
Ennesus is offline   Reply With Quote
Old 07-27-2011, 08:50 AM   #4
New Member
 
lucasbytegenius's Avatar
 
Join Date: Oct 2010
Posts: 13
Send a message via MSN to lucasbytegenius
Default

Also set text-decoration: none; in #nav a:hover.
__________________
Enigma Computer Community - A unique technology forum.
lucasbytegenius is offline   Reply With Quote
Old 07-27-2011, 09:31 AM   #5
Platinum Member
 
Join Date: Sep 2010
Posts: 509
Default

maybe a bit like this?
d a v e is offline   Reply With Quote


Old 07-27-2011, 01:31 PM   #6
New Member
 
Join Date: Jul 2011
Posts: 9
Default

Quote:
Originally Posted by Ennesus View Post
It doesn't work in Chrome for Mac.

reset your padding to the #nav and get rid of the negative margin.
Define overlow:hidden; on the nav and it will work just a little bit better.
I'll try resetting the padding. As far as defining overflow:hidden, though, will it hide part of my menu? I remember having a problem with that, but it may have just been the 'top' div that was causing problems.


d a v e - Thanks a ton for that mock-up. You have a much better eye than I do! The edges weren't antialiased because, well, I couldn't figure out how in Paint.NET or Expression Design; i'll look into it, though.

Thanks everyone!
Reed92 is offline   Reply With Quote
Old 07-27-2011, 02:16 PM   #7
Bronze Member
 
Join Date: May 2009
Posts: 49
Default

It wil hide a part of your menu when you define a height, but only when the menu is to big.In example when they're to many list items it pushes the last list item to a second row resulting the content to get moved to the right

see the following screenshot

Ennesus is offline   Reply With Quote
Old 07-27-2011, 02:45 PM   #8
Diamond Member
 
Phreaddee's Avatar
 
Join Date: Feb 2011
Location: Newcastle, Australia
Posts: 1,135
Default

  1. I think the underline on your h1 is confusing.
    either you use underlines for headers and no underlines for links (rather colour change) or vice versa, alternatively if you really wanted to keep the underline on the header than add a bottom border to the h1 rather than text-decoration. you can then style as you want, and a subtle difference will avoid confusion.
  2. Give your paragraphs some line-height! very important.
  3. font size in "x-large" etc? if you want to use a relative amount use em's, which is much easier to work with, or px if you want a fixed amount. I would make the font-size of the nav smaller so the issue shown by Ennesus doesn't rear its ugly head on the first sign of a cntrl+ (or cntrl- oddly enough), they need some room to move. I can read that from the other side of the room, yet i'm sitting right in front of it. smaller. please. (but not too small...)
  4. I would choose a colour to add into the mix, and the red headers on daves mockup appeared to brighten the mix a lot, so I would go with something like that. Also the sepia image needs to be changed to a warmer tone or neutral. your grey background also could be matched better. try http://kuler.adobe.com/#create/fromacolor you can make some awesome colour palettes from that site.
  5. change link colours from default. have a full font stack, etc etc although thats little nitpicking things.
__________________
if (headhurts == "possibly") {
alert ("keep going!");
}
else if (headhurts == "yes") {
alert ("go to sleep");
}
else if (headhurts == "damn !@#$ mofo scripts...") {
alert ("give up and have a beer!");
}
else {
alert ("watch TV");
}

Last edited by Phreaddee; 07-27-2011 at 02:57 PM.
Phreaddee is offline   Reply With Quote
Old 07-27-2011, 03:23 PM   #9
New Member
 
Join Date: Jul 2011
Posts: 9
Default

Quote:
Originally Posted by Phreaddee View Post
  1. I think the underline on your h1 is confusing.
    either you use underlines for headers and no underlines for links (rather colour change) or vice versa, alternatively if you really wanted to keep the underline on the header than add a bottom border to the h1 rather than text-decoration. you can then style as you want, and a subtle difference will avoid confusion.
  2. Give your paragraphs some line-height! very important.
  3. font size in "x-large" etc? if you want to use a relative amount use em's, which is much easier to work with, or px if you want a fixed amount. I would make the font-size of the nav smaller so the issue shown by Ennesus doesn't rear its ugly head on the first sign of a cntrl+ (or cntrl- oddly enough), they need some room to move. I can read that from the other side of the room, yet i'm sitting right in front of it. smaller. please. (but not too small...)
  4. I would choose a colour to add into the mix, and the red headers on daves mockup appeared to brighten the mix a lot, so I would go with something like that. Also the sepia image needs to be changed to a warmer tone or neutral. your grey background also could be matched better. try http://kuler.adobe.com/#create/fromacolor you can make some awesome colour palettes from that site.
  5. change link colours from default. have a full font stack, etc etc although thats little nitpicking things.
Wow!! I completely forgot about em's . Like I mentioned, I'm working in Expression Web. Sometimes it tries to be helpful and suggests things...like font sizes in x-large...so far, it hasn't been much more helpful than Notepad.

I think it's looking pretty good now. I might dull that 'call to action' box a little; it draws the eye a little too much (as in, even I have a hard time holding my focus on the content itself).

Last edited by Reed92; 07-27-2011 at 08:55 PM. Reason: Added info
Reed92 is offline   Reply With Quote
Reply

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are On



All times are GMT. The time now is 03:54 PM.


Camera Forum - Computer Forum - Web Design Forum

Powered by vBulletin® Version 3.8.7
Copyright ©2000 - 2012, vBulletin Solutions, Inc.
Content Relevant URLs by vBSEO 3.6.0 ©2011, Crawlability, Inc.