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 11 of 11
  1. #1
    Senior Coder tomws's Avatar
    Join Date
    Nov 2007
    Location
    Arkansas
    Posts
    2,644
    Thanks
    29
    Thanked 330 Times in 326 Posts

    Please review adaptarkansas.org

    http://www.adaptarkansas.org/

    There's not much to this site, but I'd like your opinions anyway, especially because I'm incompetent at matters of style.

    The page structure was requested from on high, as well as the logo. Left and right bg images are my own additions in an attempt to lessen dead space on wider monitors, but I'm uncomfortable with the right image.

    Mapping tool page text layout is different from the rest. Just haven't changed the div. The iframe content is provided from elsewhere.

  • #2
    Regular Coder BoldUlysses's Avatar
    Join Date
    Jan 2008
    Location
    Winston-Salem, NC
    Posts
    938
    Thanks
    10
    Thanked 190 Times in 187 Posts
    Likes:

    - Fluid layout. The transparent PNG at the top in particular is a nice touch.
    - Header flows into content area very nicely.
    - Validates! Huzzah!

    Could improve:

    - Footer. Seems too deep. I would make it half the height and spread the content side to side a little bit. Different color divider bar? The white seems off when the rest of the site has nice tan accents.
    - Center column width consistency. The home.php and aboutus.php center column are one width, and the maptool.php column is another, along with having lost the brown right graphical element.

    Nice site!

  • #3
    New Coder
    Join Date
    May 2008
    Location
    Orlando, FL
    Posts
    60
    Thanks
    3
    Thanked 3 Times in 3 Posts
    The design of it is overall pretty nice, especially coming from a person who is calling himself incompetent at matters of style!

    If you have time to figure out how to fix the divs which look strange at small window sizes, you might want to.

    The mapping tool page text div is larger than on the other pages.

  • #4
    Senior Coder tomws's Avatar
    Join Date
    Nov 2007
    Location
    Arkansas
    Posts
    2,644
    Thanks
    29
    Thanked 330 Times in 326 Posts
    Thanks so far. Useful info.

    msuffern: I agree about the footer. Just couldn't put my finger on the problem. Will change that at some point.

    woop: Can you explain the small window size problem?

    All: I didn't explain well enough in the original post about the mapping tool page. I converted the base template, but haven't implemented it on that page yet. Because of the requested full width iframe and toggled divs, it needs some slight additional structure. I just haven't done the rework on the divs and toggles yet to bring it into the template.

  • #5
    Master Coder
    Join Date
    Dec 2007
    Posts
    6,682
    Thanks
    436
    Thanked 890 Times in 879 Posts
    hi,

    you have css validation warning:

    http://jigsaw.w3.org/css-validator/v...rkansas.org%2F

    you can reduce the size of some pictures to increase loading speed:

    http://www.websiteoptimization.com/s...tarkansas.org/

    remove the links to w3c validators, are uselles and slow down the site.

    no meta information, keywords, description, robots, ....

    I guess that is not finish, I will try later when you add the mapping tool if you still want a revision. I don't like that I must allow access from two domain and I avoid iframe, that as a first comment.

    Quote Originally Posted by tomws View Post
    I'm incompetent at matters of style.
    seems clear that you don't know me,

    regards

  • #6
    Senior Coder gnomeontherun's Avatar
    Join Date
    Sep 2007
    Location
    Houston
    Posts
    2,846
    Thanks
    10
    Thanked 238 Times in 229 Posts
    Your home and about us pages have content only about 600 px wide or something, and the other page fill the screen better. It looks funny on a wide screen.

    The mapping page is just inconsistent with the other two pages, but I like it better because of how it lays, but its missing the squiggly swirly image.

  • #7
    Senior Coder tomws's Avatar
    Join Date
    Nov 2007
    Location
    Arkansas
    Posts
    2,644
    Thanks
    29
    Thanked 330 Times in 326 Posts
    oesxyl, all good points. Thanks.
    Quote Originally Posted by oesxyl View Post
    no meta information, keywords, description, robots, ....
    I hadn't even thought of it. Oops. I'll make a note to take care of that.

    Quote Originally Posted by oesxyl View Post
    I don't like that I must allow access from two domain and I avoid iframe, that as a first comment.
    I don't like that, either, but it's the arrangement that was made.

    jeremywilken: Yes, I'm not so sure about the narrow text div, either, but if w3schools' stats are to be believed, 70% of windows sizes are 1024 wide, and that's the target "native" width. I build on 1280 and it's not so bad full screen. But on wider screens, I expect it will look screwy. I don't know how to design around that, so I tossed in the images. I can stretch the text box, but not sure how wide it needs to be.

  • #8
    Senior Coder gnomeontherun's Avatar
    Join Date
    Sep 2007
    Location
    Houston
    Posts
    2,846
    Thanks
    10
    Thanked 238 Times in 229 Posts
    I would just let it be more fluid like the layout.

  • #9
    Master Coder
    Join Date
    Dec 2007
    Posts
    6,682
    Thanks
    436
    Thanked 890 Times in 879 Posts
    I download the page with wget and I try to play with layout and css.
    seems you have many divs:
    - #headerunder and #headerover could be merged into a single div.
    - #MainMenu can be included in #wrapper and you can get rid of #tab
    - you must 'clear: both' the float from ul >li, or use inline instead of float
    - I guess you use empty div's for spacing, a better way is to do that from css
    - try to get rid of position absolute, #bgpic-left you can make it bacground.
    - add a global reset at top of base8.css:
    Code:
    * { margin: 0; padding: 0; }
    this make browser to don't use own margin and padding for elements.

    I'm lost in what you want to do with js, so I give up,

    good luck,

    regards

  • #10
    Senior Coder tomws's Avatar
    Join Date
    Nov 2007
    Location
    Arkansas
    Posts
    2,644
    Thanks
    29
    Thanked 330 Times in 326 Posts
    Thanks! I'll incorporate some of those changes.
    Quote Originally Posted by oesxyl View Post
    I'm lost in what you want to do with js, so I give up,
    Ha! It's mostly from the first version before I knew what they wanted to do. Can (and will) be cleaned up. Thanks, again.

  • #11
    New to the CF scene
    Join Date
    Jun 2008
    Posts
    4
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Very nice website;
    the layout is very well done.
    I have to agree with the footer although.


  •  

    Posting Permissions

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