My first website.

F

FarligOpptreden

Executive Member
Joined
Mar 5, 2007
Messages
5,396
Remove the hit counter. THEN, without being funny, read up on:

Google
Bing
Yahoo

And guess what, I got to those links by googling: "how to add my site to <name of search engine>". Pretty easy, huh?
 
D

Drake2007

Expert Member
Joined
Oct 23, 2008
Messages
4,413
Look into Google advertising for your advertising budget.

I see you've designed the site on a very large monitor, keep in mind not everyone is running 21" at full res. surfers are lazy and scrolling is tedious especially when it's mostly whitespace. Good to see a contacts form, the idea of that is so your email address doesn't get spammed so remove the mailto.
 
spiderz

spiderz

Honorary Master
Joined
Mar 24, 2006
Messages
35,105
yes, make the site only as wide as the top banner. Center aligned is ok.
drop the counter.
 
Bismuth

Bismuth

Expert Member
Joined
Jun 22, 2007
Messages
3,834
Had no issue with the width of the site, and I only have a 19" monitor at max res., but maybe that helps! :D

I would also suggest removing the counter, Google, Yahoo, and the rest have instructions how to submit your site. I would also remove all that grey area underneath Events etc., and move the SAIDSA event logo into the resulting space. This would also reduce the size of the page, and therefore any required scrolling.

Note that I think it's meant to be "Obstacle", not "Opstical" course, on the "Services" page. And "adres" should be "address", on the "Home" page.

Otherwise, doesn't look bad at all.

B
 
M

midrange

Senior Member
Joined
Apr 8, 2009
Messages
727
Get rid of the tables, centre it, and give us some background colour hover change on the menu.
 
F

FarligOpptreden

Executive Member
Joined
Mar 5, 2007
Messages
5,396
Another thing - the page title is supposed to be informative and describe (in not too many words) what the page is about. Think about the title of a book or article. It's not used to provide instructions on using the page.
 
B

bawrak

Well-Known Member
Joined
Jan 29, 2009
Messages
208
Looks good...

There's another typo on the services page "conbat alley ranges" should probably be "combat alley ranges" :)

+1 on removing the counter ;-)
 
S

Schwartz

Expert Member
Joined
Feb 15, 2009
Messages
1,699
too much white space!
and I know this might be stupid but I could not find your location anywhere!
where are you guys located?
 
R

Raithlin

Executive Member
Joined
Jan 4, 2005
Messages
5,049
Not bad for a first attempt, mate. I'll list my findings in no particular order - and I'll be nice. ;)

Front page:
  • As mentioned before (I'm blocked from work, so I'm a bit late to the party :(), get rid of the tables. You are using CSS (no less than 4 includes - you may want to combine them for faster page loading), so no excuses there.
  • Use external CSS only. As above, you're linking to 4 CSS files, but then you still have inline styles. Bad.
  • You link to some large CSS files - but you're not using half of the styles. trim your files for fat.
  • Consistency. Your first heading is outside the block, the other two are inside. I would also avoid the reverse colour headings - but that's just me. :erm:
  • Remove empty divs. You have an entire section in your code for logo, but it's all blank. Get rid of it - your page will be faster (in all respects).
  • Rule number next: IDs are unique - classes are not. This comes into play when you start using javascript (jQuery in particular, but any JS), and is a standard.
  • Remove the target="_self". No need to declare what is a default.
  • I like what is being done to avoid spambots, but please consider unintrusive javascript - accessibility is at risk including code willy-nilly in your code like that.
  • Get rid of the inline CSS IN THE MIDDLE OF THE ****** PAGE! Oh my, you nearly gave me a heart attack there.:eek:
  • Get rid of the empty divs - all 28-in-a-row of them (in the middle of your source code).
  • We don't use spacers (spacer.gif). Let's just leave it at that.
  • font tags are deprecated. Use CSS classes instead.
  • &nbsp; - use padding (CSS) instead.
  • Don't use iframes if you can help it. The map doesn't display in my browser - and I think it's because my browser blocks iframes out of security concerns.
  • Empty tables. In fact, ANY empty tags should be removed. You have an entire table, complete with cells, toward the end of your front page - empty. A complete waste of bandwidth.
Right, I see you used RVSiteBuilder. That's cheating - and the reason for more than half of the list above. :p I mean, really - CSS in the middle of a body?! :erm:

I would encourage you to learn HTML and CSS at a bare minimum if you want to do this even as a hobby.

UPFRONT DISCLAIMER: (©FarligOpptreden 2009) The above text was all based on the front page of the site, before I'd even looked at the other pages.

As far as content and style goes (which I suspect may be the real "whaddya think" you're asking for), it's not bad, but not fantastic either - it screams AMATEUR :D. As I said before, skip the inverted headings. Rather use larger fonts, perhaps a different colour and/or font.

Your Services page lokos the best of the lot. Take that, and apply it to the other pages. Consistency counts for a lot more than you think.
 
You must log in or register to reply here.
Top