![]() |
|
|
|||||||
![]() |
|
|
Thread Tools | Rate Thread |
|
|
PM User | #1 |
|
Regular Coder ![]() Join Date: Mar 2008
Posts: 118
Thanks: 3
Thanked 9 Times in 9 Posts
![]() |
Business-Style Site: Please Review
I am done with the first page only, but I wanted to know what you all thought about the general look and feel. I've been using table layouts for my past few paid projects I've done for people, and now that I am finally going somewhat legit, I figured my business web site should be a table less design. So any tips with the code are very welcome.
http://legacywebdev.onlinewebshop.net/ Thank you for any comments you may have. Last edited by peteyb383; 03-22-2008 at 09:19 PM.. |
|
|
|
|
|
PM User | #2 |
|
Regular Coder ![]() Join Date: Apr 2007
Location: Quakertown PA USA
Posts: 931
Thanks: 1
Thanked 104 Times in 102 Posts
![]() |
The menu would look better if all the options were on the same line, and within the menu bar.
You'd probably be better off using a <ul> for your menu, rather than the multiple <div>s. I'd also move the menu mouseover effect to a fully css solution. Edit: BTW:
Check the site in browsers other than IE to see what I mean about the menu options. IE shows them on one line. Firefox, Safari (pc) & Opera all show 'Contact Us' on a separate line in the body area. Last edited by PappaJohn; 03-22-2008 at 05:46 AM.. |
|
|
|
|
|
PM User | #7 |
|
Senior Coder ![]() ![]() Join Date: Dec 2007
Posts: 4,658
Thanks: 374
Thanked 562 Times in 551 Posts
![]() ![]() |
- add a css global reset to the page:
Code:
* { margin: 0; padding: 0; }
- declare a font, now this break your layout - you make the width of the browser fixed, deal somehow with font setings - use overflow or clear to deal with float element - the menu, as PappaJohn said, make it css - on my screen "contact us" is few pixels under "Need a developer" best regards |
|
|
|
|
|
PM User | #8 |
|
Regular Coder ![]() Join Date: Mar 2008
Posts: 118
Thanks: 3
Thanked 9 Times in 9 Posts
![]() |
I changed the link for the site as my other free hosting server went down.
http://legacywebdev.onlinewebshop.net/ I'm getting a paid hosting account soon and an actual domain name as well, but for now, all I have to use is free hosting and subdomains. I changed the navigation and the background. Tell me what you think. EDIT: Agh... IE7, Firefox, and Netscape render it alright, but IE6 still butchers it completely. I don't know what to do. Last edited by peteyb383; 03-22-2008 at 09:21 PM.. |
|
|
|
|
|
PM User | #9 |
|
Regular Coder ![]() Join Date: Apr 2007
Location: Quakertown PA USA
Posts: 931
Thanks: 1
Thanked 104 Times in 102 Posts
![]() |
A definite improvement. I think I would darken the menu text a bit. Especially on Firefox, there isn't enough contrast between the text color & the background.
I'm assuming the id="active" on the 'Home' <li> is intended to visually distinguish it from the other menu items, but I'm seeing no difference.You are likely going to have to look into some of the IE6 css hacks to make it play nice. |
|
|
|
|
|
PM User | #10 |
|
Regular Coder ![]() Join Date: Mar 2008
Posts: 118
Thanks: 3
Thanked 9 Times in 9 Posts
![]() |
Yeah, I forgot to implement the active id in the CSS, but I'm debating on whether to put it in or not.
As for CSS hacks in IE6, It would be a much better world if everyone just used Firefox... |
|
|
|
![]() |
| Bookmarks |
| Thread Tools | |
| Rate This Thread | |
|
|