Go Back   CodingForums.com > :: Client side development > General web building > Site reviews

Before you post, read our: Rules & Posting Guidelines

Reply
 
Thread Tools Rate Thread
Enjoy an ad free experience by logging in. Not a member yet? Register.
Old 07-14-2010, 01:31 AM   PM User | #1
thomasmclennan
New Coder

 
Join Date: Jul 2010
Posts: 33
Thanks: 2
Thanked 1 Time in 1 Post
thomasmclennan is an unknown quantity at this point
Cool Sports Club Website

I am just finishing up a website for my short track speed skating club. It's almost completed, only a few more pages need content, etc.

http://ridgemeadowsracers.org


Any reviews comments and especially criticism would be welcome =)

Also, The few people that have looked at it commented that some of the site was hard to navigate (eg: I added a text link to the home page under the logo, because it wasn't obvious enough).

A comment on CSS validation. It brings up four errors, eg:
Code:
	 Value Error : font-family Property font-family doesn't exist in CSS level 2.1
for the font only used on the home page. I believe that it is semantic in CSS 3. The only reason it is (the code) there, is that otherwise, it is not cross-browser. The bad CSS code is:
Code:
@font-face {	font-family: Addcityboy;src: url('/lib/design_fonts/Addcityboy.eot');}
@font-face {	font-family: Addcityboy;src: url('lib/design_fonts/Addcityboy.TTF');}
Thank you for your time!

Last edited by thomasmclennan; 07-14-2010 at 01:37 AM..
thomasmclennan is offline   Reply With Quote
Old 07-16-2010, 07:45 PM   PM User | #2
rickzwebz
New Coder

 
Join Date: Jul 2010
Location: Winnipeg, MB, CA
Posts: 13
Thanks: 0
Thanked 0 Times in 0 Posts
rickzwebz is an unknown quantity at this point
- The light grey text over images is hard to read
- the text need more padding so the text does not run right up to the edge
- using the forum link button lost your web site

- appears you used a fluid design and did not check the site in 1024x768 which is the most common screen size. On the home page the movie is bumped to the bottom of the page

- not a fan of not having the home link with the rest of the menu

- some images that are shot in portrait display as landscape
rickzwebz is offline   Reply With Quote
Old 07-17-2010, 07:12 AM   PM User | #3
thomasmclennan
New Coder

 
Join Date: Jul 2010
Posts: 33
Thanks: 2
Thanked 1 Time in 1 Post
thomasmclennan is an unknown quantity at this point
Quote:
Originally Posted by rickzwebz View Post
- The light grey text over images is hard to read

Quote:
Originally Posted by rickzwebz View Post
- the text need more padding so the text does not run right up to the edge
Agreed, I switched it to 10px on both sides.
Quote:
Originally Posted by rickzwebz View Post
- using the forum link button lost your web site
I am thinking of putting the form into an iFrame, or at least going into the vBullitan code and changing it. I am about to get around to it after the final few pages of content are uploaded.
Quote:
Originally Posted by rickzwebz View Post
- appears you used a fluid design and did not check the site in 1024x768 which is the most common screen size. On the home page the movie is bumped to the bottom of the page
I'll re-render the video in 320x200px tonight. Though I am gathering that as technology continues to increase, screen sizes will to. Though, windows XP shipped with the resolution set to 1024 as default (?). Here is the distribution of browser resolutions that access the web, according to w3schools:


Quote:
Originally Posted by rickzwebz View Post
- not a fan of not having the home link with the rest of the menu
Quote:
Originally Posted by rickzwebz View Post
- some images that are shot in portrait display as landscape
thomasmclennan is offline   Reply With Quote
Old 07-17-2010, 07:52 PM   PM User | #4
Apostropartheid
The Apostate


 
Apostropartheid's Avatar
 
Join Date: Oct 2007
Posts: 3,215
Thanks: 16
Thanked 265 Times in 263 Posts
Apostropartheid is on a distinguished road
Still needs more padding.
What on god's green earth happened with those fonts? It's barely readable!
Your video is wider than the middle section and appears below the sidebars for me. On that topic, why are your sidebars so big?
1024px wide is the standard baseline, meaning your design should fit comfortably into that. As not everybody browses full-screen, this means you should have a maximum width of something in the 900s (personally, I author to 920px.)
__________________
Blog | Twitter
Useful links: W3C HTML Validator | W3C CSS Validator | HTML 5 Guide
CF: HTML & CSS Resources/Tutorials Thread | HTML & CSS Posting Rules and Guidelines
Remember: no link, no code, no help!
Apostropartheid is offline   Reply With Quote
Old 07-18-2010, 06:25 AM   PM User | #5
thomasmclennan
New Coder

 
Join Date: Jul 2010
Posts: 33
Thanks: 2
Thanked 1 Time in 1 Post
thomasmclennan is an unknown quantity at this point
I'm using 0.875em (14px) font for the site, though it seems that it would be best to use 12px and free up some space, thanks.
thomasmclennan is offline   Reply With Quote
Old 07-18-2010, 06:41 PM   PM User | #6
Apostropartheid
The Apostate


 
Apostropartheid's Avatar
 
Join Date: Oct 2007
Posts: 3,215
Thanks: 16
Thanked 265 Times in 263 Posts
Apostropartheid is on a distinguished road
No, it's not the size. It was just very thin and blurry on Windows with ClearType. Did you check machines with ClearType enabled?
__________________
Blog | Twitter
Useful links: W3C HTML Validator | W3C CSS Validator | HTML 5 Guide
CF: HTML & CSS Resources/Tutorials Thread | HTML & CSS Posting Rules and Guidelines
Remember: no link, no code, no help!
Apostropartheid is offline   Reply With Quote
Old 07-18-2010, 06:45 PM   PM User | #7
thomasmclennan
New Coder

 
Join Date: Jul 2010
Posts: 33
Thanks: 2
Thanked 1 Time in 1 Post
thomasmclennan is an unknown quantity at this point
Quote:
Originally Posted by Apostropartheid View Post
No, it's not the size. It was just very thin and blurry on Windows with ClearType. Did you check machines with ClearType enabled?
No, I have only had a chance to check it on a Vista machine and my iPhone.
Most of the font uses
Code:
font-family:"Helvetica Neue LT Com 33", "HelveticaNeue-Light", "Helvetica Neue Light", "Helvetica Neue", "Helvetica", Verdana, Geneva, sans-serif;
, maybe I should add Arial in there.
thomasmclennan is offline   Reply With Quote
Reply

Bookmarks

Tags
@font-face, css, review

Jump To Top of Thread


Thread Tools
Rate This Thread
Rate This Thread:

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

Forum Jump


All times are GMT +1. The time now is 11:11 AM.


Advertisement
Log in to turn off these ads.