8 replies [Last post]
TheEvilDale
Offline
newbie
Last seen: 15 years 29 weeks ago
Joined: 2006-05-14
Posts: 4
Points: 0

Hi, I was hoping to get some feedback on my new design.
http://forboden.com

Cheers.

Tyssen
Tyssen's picture
Offline
Moderator
Brisbane
Last seen: 7 years 7 weeks ago
Brisbane
Timezone: GMT+10
Joined: 2004-05-01
Posts: 8201
Points: 1386

Another site review

I think your links in the header are too dark. The site doesn't resize well down to 800 x 600 and you could do with using proportional units for the margins on the elements in your header as they start to overlap when you resize the text.

How to get help
Post a link. If you can't post a link, jsFiddle it.
My blog | My older articles | CSS Reference

TheEvilDale
Offline
newbie
Last seen: 15 years 29 weeks ago
Joined: 2006-05-14
Posts: 4
Points: 0

Another site review

Tyssen wrote:
I think your links in the header are too dark...you could do with using proportional units for the margins on the elements in your header as they start to overlap when you resize the text.

Thanks, I'll adjust those two things.

Quote:
The site doesn't resize well down to 800 x 600 and

I found it impossible to make the floats on the index page evenly spaced when I made the page fluid. Since it's Wordpress, there's no way (that I could see), to give a different id or class to the floated divs on the right.

Tyssen
Tyssen's picture
Offline
Moderator
Brisbane
Last seen: 7 years 7 weeks ago
Brisbane
Timezone: GMT+10
Joined: 2004-05-01
Posts: 8201
Points: 1386

Another site review

Do you mean giving every second record pulled from the db a different class? You'd do something like this:

<?php while (have_posts()) : the_post();
if($count == 0)	{
$count = $count + 1;
$post_class = "style1";
} else {
$count = 0;
$post_class = "style2";
} ?>
<div class="<?php echo $post_class; ?>">

How to get help
Post a link. If you can't post a link, jsFiddle it.
My blog | My older articles | CSS Reference

TheEvilDale
Offline
newbie
Last seen: 15 years 29 weeks ago
Joined: 2006-05-14
Posts: 4
Points: 0

Another site review

Tyssen wrote:
You'd do something like this:

That's awesome. It worked perfectly. Laughing out loud

Would you mind if I posted that code on the wordpress forums? I'm sure others would find it useful.

Tyssen
Tyssen's picture
Offline
Moderator
Brisbane
Last seen: 7 years 7 weeks ago
Brisbane
Timezone: GMT+10
Joined: 2004-05-01
Posts: 8201
Points: 1386

Another site review

TheEvilDale wrote:
Would you mind if I posted that code on the wordpress forums? I'm sure others would find it useful.

I think I probably got it from there in the first place. Wink

How to get help
Post a link. If you can't post a link, jsFiddle it.
My blog | My older articles | CSS Reference

technossomy
technossomy's picture
Offline
Enthusiast
Last seen: 8 weeks 4 days ago
Timezone: GMT+2
Joined: 2004-06-09
Posts: 260
Points: 8

Another site review

There is a lengthy debate on this coloring issue to be found here:
http://www.csscreator.com/css-forum/ftopic18502.html

zhastro
zhastro's picture
Offline
Enthusiast
Indonesia
Last seen: 10 years 25 weeks ago
Indonesia
Timezone: GMT+7
Joined: 2005-10-22
Posts: 73
Points: 0

Another site review

too simple and too dark for me....

Only You Can Reach Me Now....
Only You Can Teach Me Now...
Teach Me How To Dream...

ABCruz
ABCruz's picture
Offline
Enthusiast
Pilipinas
Last seen: 7 years 30 weeks ago
Pilipinas
Timezone: GMT+8
Joined: 2005-06-16
Posts: 279
Points: 0

Another site review

^I agree that it's too simple although I don't mind it being dark (but that's just me).

I am a Man of Truth.
I am a Free Human Person.
I am a Peacemaker.

http://www.abcruz.com