15 replies [Last post]
Ryan
Ryan's picture
Offline
newbie
Aberdeen, Scotland
Last seen: 17 years 22 weeks ago
Aberdeen, Scotland
Joined: 2005-08-24
Posts: 10
Points: 0

Hi there, I've been learning web design for about 9 months and recently released a portfolio site for my past and future work. I'd like it if people could crit the css / xhtml of the site - www.thevisualprocess.com -

Thanks.

[edit] it has not been viewed let alone tested on a mac yet, I have one in the mail on its way to me right now though Wink

thepineapplehead
thepineapplehead's picture
Offline
Moderator
Last seen: 39 weeks 5 days ago
Timezone: GMT+1
Joined: 2004-06-30
Posts: 9683
Points: 819

Please crit my site

First impression - it's too narrow. I have a 17inch CRT at 1152 and there's about 5 inches of . . . nothingness on either side. Wasted space, imho.

I like the style, very clean.

The dark green text could do with being a little darker, to make it easier to read. And increase the font size slightly - but only of the green content text - if you resize the pinky text it would ruin it.

And did you know Ormtec's background image repeats? Notch up your resolution. It's horribly noticeable Laughing out loud

Verschwindende wrote:
  • CSS doesn't make pies

Ryan
Ryan's picture
Offline
newbie
Aberdeen, Scotland
Last seen: 17 years 22 weeks ago
Aberdeen, Scotland
Joined: 2005-08-24
Posts: 10
Points: 0

Please crit my site

thepineapplehead wrote:
First impression - it's too narrow. I have a 17inch CRT at 1152 and there's about 5 inches of . . . nothingness on either side. Wasted space, imho.

hmmm, :-k I'll see what I can do about that. It was wider and had a sidebar but because there is such a small amount of content and no need for a sidebar I cut it out and made it slimmer. I'll take your point onboard though.

Quote:
I like the style, very clean.

The dark green text could do with being a little darker, to make it easier to read. And increase the font size slightly - but only of the green content text - if you resize the pinky text it would ruin it.
Do you mean the dark green text of the body or the nav? I think I'll make the nav slightly darker and notch up the darkness on the body text. Smile

Quote:
And did you know Ormtec's background image repeats? Notch up your resolution. It's horribly noticeable Laughing out loud

oh that has nothing to do with me Wink I'm designing a different website for one of their products. Maybe once thats done I'll suggest a redesign of their main site heh.

[edit] thats the color slightly darkened in the body text and nav buttons. Also the text size has been increased to 1.1em (11px). I think I'll add in an option for the user to change the text size.

thatsgame1
thatsgame1's picture
Offline
Enthusiast
Tallahassee, FL
Last seen: 15 years 3 weeks ago
Tallahassee, FL
Joined: 2004-11-06
Posts: 69
Points: 0

Please crit my site

thepineapplehead what's another way to do the background image without repeating? Is there a certain image size you use? Thanks for the help.

Ryan
Ryan's picture
Offline
newbie
Aberdeen, Scotland
Last seen: 17 years 22 weeks ago
Aberdeen, Scotland
Joined: 2005-08-24
Posts: 10
Points: 0

Please crit my site

You'd need to do this:

background: url(images/my_bg_img.jpg) no-repeat;

thatsgame1
thatsgame1's picture
Offline
Enthusiast
Tallahassee, FL
Last seen: 15 years 3 weeks ago
Tallahassee, FL
Joined: 2004-11-06
Posts: 69
Points: 0

Please crit my site

Ryan,

Does that work if your background picture is say 10 pixels wide? Otherwise, how would you get the same affect at a repeating background?

Andrew

gleddy
gleddy's picture
Offline
Leader
sydney, australia
Last seen: 14 years 38 weeks ago
sydney, australia
Timezone: GMT+10
Joined: 2004-09-21
Posts: 596
Points: 0

Please crit my site

I like with styles off... well thought out and good use of <hr>'s to seperate all the content.

maybe to narrow as previously mentioned? You could try a few variations of width.

DeprecatedDiva
DeprecatedDiva's picture
Offline
Enthusiast
NW Louisiana
Last seen: 16 years 7 weeks ago
NW Louisiana
Timezone: GMT-6
Joined: 2005-06-12
Posts: 135
Points: 0

Please crit my site

The graphics used in "News" beg to be clickable links. I don't know if this is a professionally sound idea or not but my first inclination was to click on the image to go visit the listed sites.

DeprecatedDiva

gleddy
gleddy's picture
Offline
Leader
sydney, australia
Last seen: 14 years 38 weeks ago
sydney, australia
Timezone: GMT+10
Joined: 2004-09-21
Posts: 596
Points: 0

Please crit my site

true eh?? that could be something to look at... having links and non-links the same colour is a bit of a confusion.

thepineapplehead
thepineapplehead's picture
Offline
Moderator
Last seen: 39 weeks 5 days ago
Timezone: GMT+1
Joined: 2004-06-30
Posts: 9683
Points: 819

Please crit my site

Gotta agree - in each news section, the title and image shoudl be links as well.

Much better with the color and font size though Laughing out loud

Verschwindende wrote:
  • CSS doesn't make pies

conoobus
Offline
Regular
Last seen: 17 years 23 weeks ago
Timezone: GMT+1
Joined: 2005-08-21
Posts: 28
Points: 0

Please crit my site

XTHML eh?

Anonymous
Anonymous's picture
Guru

Please crit my site

conoobus wrote:
XTHML eh?
Good catch. Laughing out loud

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

Please crit my site

Nothing to do with your design (which I like just fine) but one thing you might want to consider about your content is that you've used all the current 'catchphrases' to describe what you do, your skills etc., but for the most part, these only mean anything to other designers/people in the business.
I've just finished doing my own portfolio/business site and it was something I thought about when I was writing my content and I added some extra information about XHTML/CSS and what it is and why it is a good thing for those people who know nothing about how websites are created.

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

Anonymous
Anonymous's picture
Guru

Please crit my site

Tyssen wrote:
Nothing to do with your design (which I like just fine) but one thing you might want to consider about your content is that you've used all the current 'catchphrases' to describe what you do, your skills etc., but for the most part, these only mean anything to other designers/people in the business.
This is a good point. Anyone that knows what you are talking about probably does it for themselves. Smile

conoobus
Offline
Regular
Last seen: 17 years 23 weeks ago
Timezone: GMT+1
Joined: 2005-08-21
Posts: 28
Points: 0

Please crit my site

also content related...
you have a skills assement of yourself w/ various programs/mark-ups.

I'm not so sure this is a good idea.

Your clients don't need to know that you aren't that proficient in Quark, just that you're familiar w/ it. You certainly want to emphasize what you know well, but not advertise your ignorance.

Anonymous
Anonymous's picture
Guru

Please crit my site

conoobus wrote:
also content related...
you have a skills assement of yourself w/ various programs/mark-ups.

I'm not so sure this is a good idea.

Your clients don't need to know that you aren't that proficient in Quark, just that you're familiar w/ it. You certainly want to emphasize what you know well, but not advertise your ignorance.It is a nice use of the list item background image though. Smile