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

Before you post, read our: Rules & Posting Guidelines

View Poll Results: What do u think of my site?
I love it 1 8.33%
coool 1 8.33%
ah its okay 6 50.00%
I hate 4 33.33%
Voters: 12. You may not vote on this poll

Reply
 
Thread Tools Rate Thread
Enjoy an ad free experience by logging in. Not a member yet? Register.
Old 01-17-2004, 08:43 PM   PM User | #16
lnxusr86
New Coder

 
Join Date: Nov 2003
Posts: 61
Thanks: 0
Thanked 0 Times in 0 Posts
lnxusr86 is an unknown quantity at this point
any other things before i begin redesigning?
lnxusr86 is offline   Reply With Quote
Old 01-18-2004, 12:13 AM   PM User | #17
mindlessLemming
Senior Coder

 
Join Date: Oct 2003
Location: Australia
Posts: 1,963
Thanks: 0
Thanked 0 Times in 0 Posts
mindlessLemming is an unknown quantity at this point
Well, besides your code, which I feel relies on javascript for absolutely no reason, (other than your beloved pre-loader ) -
Your navigation is quite tired and dull, as well as being a jumbled mess of tables and js calls without any option for those with javascript disabled, such as myself.

It would far more managable/accessible/less code/better looking
if you used an unordered list <ul>with links <a> nested inside the list elements <li>. You can see a very basic example on my links site , and if you go to the CSS links section there are far better examples from places like a list apart, SimpleBits and others.

Also, when you redo your nav. with <ul> and CSS, put equal padding on left and right of <a> text so as to centre the text within it's display block. Perhaps a 1px solid border around each button slightly darker than its background as well as a narrow margin to seperate each would make them more aestheticly pleasing.
And don't forget to assign a different colour to a:hover{color:#......}
so the link text changes colour on hover. This is an important visual clue that should not be removed.

!important
If you really want to keep the sliced images and lay the page out in the traditional way, use Adobe ImageReady 7 for the slicing/exporting.
It has options which allow you to get it to make all slices <div>'s and all styling CSS. This is a fantastic option for those trying to make the transition from chopped up tables to flowing em's or wherever.
Just make sure you set it to ALL LOWERCASE tags and attributes and to make the slices background images of their parent div, allowing you to freely add other elements into the div later.

SOme of this may not have made much sense, but I wrote the second half assuming you had gone and done everything I recommended in the first half

Have fun redesigning,
Andrew

**edit: Image Ready 7. Don't remember if 3 could as well. (ver.3 shipped with Photoshop 6)
__________________

I take no responsibility for the above nonsense.


Left Justified

Last edited by mindlessLemming; 01-18-2004 at 04:22 AM..
mindlessLemming is offline   Reply With Quote
Reply

Bookmarks

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 08:44 AM.


Advertisement
Log in to turn off these ads.