Jump to content


Photo

New Small Business Site


  • This topic is locked This topic is locked
17 replies to this topic

#1 AV1611

AV1611
  • Members
  • PipPipPip
  • Advanced Member
  • 997 posts

Posted 26 March 2006 - 08:47 PM

critiques and suggestions please:

[a href=\"http://www.blmservices.com\" target=\"_blank\"]www.blmservices.com[/a]

Thanks!

#2 ober

ober
  • Staff Alumni
  • Advanced Member
  • 5,337 posts
  • LocationEast Coast, USA

Posted 27 March 2006 - 04:42 PM

I'd do some work on the nav... drop the images and use more CSS. Or at least pad the top a little so the words are centered vertically.

I also would move the left picture down a line or two. It looks off a little when you only have one line above it.

And lastly, the whole site is kinda bland as far as color. You need something that stands out.

Info: PHP Manual


#3 AV1611

AV1611
  • Members
  • PipPipPip
  • Advanced Member
  • 997 posts

Posted 27 March 2006 - 05:26 PM

[!--quoteo(post=358941:date=Mar 27 2006, 11:42 AM:name=ober)--][div class=\'quotetop\']QUOTE(ober @ Mar 27 2006, 11:42 AM) View Post[/div][div class=\'quotemain\'][!--quotec--]
I'd do some work on the nav... drop the images and use more CSS.




Or at least pad the top a little so the words are centered vertically.
// I'll pad the words so they center on the buttons better...

I also would move the left picture down a line or two.
// I'll do that too...

It looks off a little when you only have one line above it.

And lastly, the whole site is kinda bland as far as color.
// The two pictures were in color, but I thought they clashed a little.. I'll change them back so you can tell me if it looks better with the color back in them...

Give me some time to change the pics back to color so you can tell me what you think...


You need something that stands out.
[/quote]


#4 AV1611

AV1611
  • Members
  • PipPipPip
  • Advanced Member
  • 997 posts

Posted 11 April 2006 - 02:21 AM

Ok, made some changes (centered words, colored pictures, and other little things) let me know what you think.

[a href=\"http://www.BLMServices.com\" target=\"_blank\"]http://www.BLMServices.com[/a]

#5 steviewdr

steviewdr
  • Moderators
  • Advanced Member
  • 1,364 posts
  • LocationIreland

Posted 11 April 2006 - 10:45 AM

I think it needs a little more edge definition.

I would put a 1px black border on the right and left of the main div - it would focus the eye more on the content.

Also - for the images - I would put a 1px black border around the images.

A proper footer div would be good also - and give a little more colour to the bottom.

-steve

#6 moberemk

moberemk
  • Members
  • PipPipPip
  • Advanced Member
  • 695 posts

Posted 11 April 2006 - 11:14 AM

The text of your upper nav is hard to read on those images. Your code is a hideous mismatch of uppercasing and lowecasing. And last, but far from least, there is no other pages to look at.
Disclaimer: No matter how harsh I may be, I'm always IMO only.

#7 AV1611

AV1611
  • Members
  • PipPipPip
  • Advanced Member
  • 997 posts

Posted 11 April 2006 - 01:53 PM

Help me understand:

Is the mix of upper and lower case in code, other than being lazy, a problem for any particular browser? or is it just "good practice" issue?

BTW: Mober, are you like Obers evil twin??? LOL

#8 wildteen88

wildteen88
  • Staff Alumni
  • Advanced Member
  • 10,482 posts
  • LocationUK, Bournemouth

Posted 11 April 2006 - 02:08 PM

You shouldn't really put parts of your code in uppercase then some areas in lowercase as it makes the code look untidy just stick with one or the other. I prefer to have all my code in lowercase and indented. Such as:
<div>
  <p>Hello there</p>
  <h1>blah blah</h1>
  <p>what you doing?<br />
    I'm fine you?<br />
    Not too bad myself!
  </p>

  <div>
    Hello
    <div>Something else</div>
  </div>
</div>

[!--quoteo--][div class=\'quotetop\']QUOTE[/div][div class=\'quotemain\'][!--quotec--]BTW: Mober, are you like Obers evil twin??? LOL[/quote]LMAO of with that great comment.

#9 AndyB

AndyB
  • Staff Alumni
  • Advanced Member
  • 5,465 posts
  • LocationToronto

Posted 11 April 2006 - 02:08 PM

[!--quoteo(post=363651:date=Apr 11 2006, 09:53 AM:name=AV1611)--][div class=\'quotetop\']QUOTE(AV1611 @ Apr 11 2006, 09:53 AM) View Post[/div][div class=\'quotemain\'][!--quotec--]
Help me understand:

Is the mix of upper and lower case in code, other than being lazy, a problem for any particular browser? or is it just "good practice" issue?[/quote]

Standards, standards, standards ... XHTML documents must use lower case for all HTML element and attribute names.

Legend has it that reading the manual never killed anyone.
My site

#10 ober

ober
  • Staff Alumni
  • Advanced Member
  • 5,337 posts
  • LocationEast Coast, USA

Posted 11 April 2006 - 02:22 PM

Andy is right here. Definately stick to lowercase.

And maybe I'm moberemk's evil twin, eh?

Info: PHP Manual


#11 AV1611

AV1611
  • Members
  • PipPipPip
  • Advanced Member
  • 997 posts

Posted 11 April 2006 - 05:56 PM

Mobe, you win, I'll be sure to keep small case whenever possible, and thanks for pointing that out. Ober, thanks for the good laugh at the cost of your impecable name. thanks everyone else, too!

BTW, does anyone else agree with Mobe's comment:

[!--quoteo--][div class=\'quotetop\']QUOTE[/div][div class=\'quotemain\'][!--quotec--]
The text of your upper nav is hard to read on those images.
[/quote]

I thought the looked good, but I'm biased LOL...

#12 moberemk

moberemk
  • Members
  • PipPipPip
  • Advanced Member
  • 695 posts

Posted 11 April 2006 - 07:11 PM

Ober's evil twin, hmm? Ironically, that's one of the nicer things people have said to me!
That being said, I definitely agree! But then, it is my comment after all...
Disclaimer: No matter how harsh I may be, I'm always IMO only.

#13 ober

ober
  • Staff Alumni
  • Advanced Member
  • 5,337 posts
  • LocationEast Coast, USA

Posted 11 April 2006 - 07:14 PM

I do agree that the main nav is hard to read with those images. You may want to tone them down a bit or swap them out for something else.

And moberemk, after all the things people have said negatively about me here, I'm not sure you want to be associated with me ;-)

Info: PHP Manual


#14 AV1611

AV1611
  • Members
  • PipPipPip
  • Advanced Member
  • 997 posts

Posted 13 April 2006 - 01:32 AM

Are the nav buttons better now?

[a href=\"http://www.blmservices.com\" target=\"_blank\"]LINK[/a]

#15 AndyB

AndyB
  • Staff Alumni
  • Advanced Member
  • 5,465 posts
  • LocationToronto

Posted 13 April 2006 - 01:51 AM

The nav buttons are better now. Still ugly, but they're better.
Legend has it that reading the manual never killed anyone.
My site

#16 AV1611

AV1611
  • Members
  • PipPipPip
  • Advanced Member
  • 997 posts

Posted 13 April 2006 - 07:39 PM

I am usually a good judge of ugly, they just don't seem ugly to me... is it the color contrast?


#17 redbullmarky

redbullmarky
  • Staff Alumni
  • Advanced Member
  • 2,863 posts
  • LocationBedfordshire, England

Posted 13 April 2006 - 09:39 PM

[!--quoteo(post=364533:date=Apr 13 2006, 08:39 PM:name=AV1611)--][div class=\'quotetop\']QUOTE(AV1611 @ Apr 13 2006, 08:39 PM) View Post[/div][div class=\'quotemain\'][!--quotec--]
I am usually a good judge of ugly, they just don't seem ugly to me... is it the color contrast?
[/quote]

i wouldnt say ugly. more 'confused'. on one hand, they're nearly simple, on the other hand they're nearly 'polished'. but because they're neither one nor the other, it just dont look right. either polish them up (cleaner 3D effect) or strip them right back to nice, simple and clean CSS rollovers.
"you have to keep pissing in the wind to learn how to keep your shoes dry..."

I say old chap, that is rather amusing!

#18 AV1611

AV1611
  • Members
  • PipPipPip
  • Advanced Member
  • 997 posts

Posted 14 April 2006 - 02:23 PM

Ahh... That makes sense...




0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users