Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Results 1 to 10 of 10
  1. #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 08:19 PM.

  • #2
    Senior Coder
    Join Date
    Apr 2007
    Location
    Quakertown PA USA
    Posts
    1,028
    Thanks
    1
    Thanked 125 Times in 123 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 04:46 AM.

  • #3
    Regular Coder
    Join Date
    Mar 2008
    Posts
    118
    Thanks
    3
    Thanked 9 Times in 9 Posts
    Hmm... What resolution are you on? I checked it on Firefox, Netscape, and IE and they were all on one line.

    Thanks for the tips though, I will put them into effect.

  • #4
    Senior Coder
    Join Date
    Apr 2007
    Location
    Quakertown PA USA
    Posts
    1,028
    Thanks
    1
    Thanked 125 Times in 123 Posts
    Much more than is needed by the site - 1440 x 900

  • #5
    Regular Coder
    Join Date
    Mar 2008
    Posts
    118
    Thanks
    3
    Thanked 9 Times in 9 Posts
    Strange, I'm running 1280 x 1024 and it shows up fine, no matter though, I'm going to use an unordered list.

  • #6
    Senior Coder
    Join Date
    Apr 2007
    Location
    Quakertown PA USA
    Posts
    1,028
    Thanks
    1
    Thanked 125 Times in 123 Posts
    You also might want to take a look at it in IE 6 - it makes a mess of it.

  • #7
    Master Coder
    Join Date
    Dec 2007
    Posts
    6,682
    Thanks
    436
    Thanked 890 Times in 879 Posts
    - add a css global reset to the page:
    Code:
    * { margin: 0; padding: 0; }
    put inside what you need for common browsers
    - 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

  • #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 08:21 PM.

  • #9
    Senior Coder
    Join Date
    Apr 2007
    Location
    Quakertown PA USA
    Posts
    1,028
    Thanks
    1
    Thanked 125 Times in 123 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.

  • #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...


  •  

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •