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 15 of 15
  1. #1
    Codeasaurus Rex
    Join Date
    Jun 2008
    Location
    Redmond, WA
    Posts
    659
    Thanks
    31
    Thanked 100 Times in 94 Posts

    Time for my review

    Ladies and gentlemen, boys and girls: Please review my website.

    KNOWN ISSUES Working on addressing them
    -IE6- Compatibility
    -PNG Compatibility (In Pre-PNG browsers)
    -Javascript disabled = Open new page for some dynamic content (At this point intended, I may re-work though)

    UNKOWN ISSUES
    -Any ideas?

    NON-404 PAGES
    I have not recieved content on quite a few pages so I've put dummy text on only a few. All content pages will follow the same basic scheme
    -Home
    -About Us
    -Issues
    -Issues -> Civil Rights
    -Campaigns
    -Outreach
    -Support our cause (Place holder page)

    Without further adieu,

    http://cr.xt-arts.com

    Many thanks for any praises and/or criticisms. Please feel free to scold my lifestyle, trounce on my beliefs, and make me cry - but please do it in a polite way .

    Regards
    Last edited by ShaneC; 07-23-2008 at 09:31 PM.
    Unless otherwise stated, any code posted is most likely untested and may contain syntax errors.
    My posts, comments, code, and suggestions reflect only my personal views.
    Web Portfolio and Code Snippets: http://shanechism.com

  • #2
    Regular Coder
    Join Date
    May 2008
    Location
    Lost in Localhost...
    Posts
    702
    Thanks
    3
    Thanked 43 Times in 42 Posts
    The MAJOR thing that goes wrong is the menu.
    The "Outreach" "Blog" "Contact" and "Log into myCR" are under the other menu options, and when the drop down menu drops down, it screws everything up and its completely un-usable.

    And on the 404 error pages, the same menu items fall down, and push the "404 error" image to the right and make it stick out.

    If those navigation items were fixed it'd look really nice. But at the moment, it's completely un-usable for me.

    (I use Firefox 3 with Linux fonts)

  • Users who have thanked Millenia for this post:

    ShaneC (07-23-2008)

  • #3
    Codeasaurus Rex
    Join Date
    Jun 2008
    Location
    Redmond, WA
    Posts
    659
    Thanks
    31
    Thanked 100 Times in 94 Posts
    Quote Originally Posted by Millenia View Post
    The MAJOR thing that goes wrong is the menu.
    The "Outreach" "Blog" "Contact" and "Log into myCR" are under the other menu options, and when the drop down menu drops down, it screws everything up and its completely un-usable.

    And on the 404 error pages, the same menu items fall down, and push the "404 error" image to the right and make it stick out.
    Please post your browser specifications. I have opened in Safari, IE 7, Firefox 2 & 3 and I'm unable to re-produce that error

    Edit: LOL. *Hits head* I uploaded a wrong header file - my bad FF3 users! Try again please
    Unless otherwise stated, any code posted is most likely untested and may contain syntax errors.
    My posts, comments, code, and suggestions reflect only my personal views.
    Web Portfolio and Code Snippets: http://shanechism.com

  • #4
    Regular Coder
    Join Date
    May 2008
    Location
    Lost in Localhost...
    Posts
    702
    Thanks
    3
    Thanked 43 Times in 42 Posts
    LOL. *Hits head* I uploaded a wrong header file - my bad FF3 users! Try again please
    Glad you fixed it, I went back onto it and I was like eh...

    Everything you want to stand out, stands out.
    Navigation is great to use.
    Repetitive use of the logo.
    and as I see...fast loading.
    And valid!

    Nice job.

    Wait, is the footer meant to be hard to read? Because if I were you I would change it to white/cream or something
    Last edited by Millenia; 07-23-2008 at 09:18 PM. Reason: Wait, is the footer meant to be hard to read? Because if I were you I would change it to white/cream or something

  • #5
    Codeasaurus Rex
    Join Date
    Jun 2008
    Location
    Redmond, WA
    Posts
    659
    Thanks
    31
    Thanked 100 Times in 94 Posts
    Quote Originally Posted by Millenia View Post
    Wait, is the footer meant to be hard to read? Because if I were you I would change it to white/cream or something
    Actually yeah it is. I've never been a huge fan of really intrusive legal-text so I just kind of stuck it there ninja-like. It will be changed to different text post launch anyway.

    Good catch, though, and thanks for your input!
    Unless otherwise stated, any code posted is most likely untested and may contain syntax errors.
    My posts, comments, code, and suggestions reflect only my personal views.
    Web Portfolio and Code Snippets: http://shanechism.com

  • #6
    Master Coder
    Join Date
    Dec 2007
    Posts
    6,682
    Thanks
    436
    Thanked 890 Times in 879 Posts
    1. I like it
    2. not a problem but have some warnings:
    http://jigsaw.w3.org/css-validator/v...xt-arts.com%2F
    3. that is a problem:
    http://www.websiteoptimization.com/s...r.xt-arts.com/
    at least jpg used for layout can have smaller size and load faster.

    regards

  • Users who have thanked oesxyl for this post:

    ShaneC (07-23-2008)

  • #7
    Codeasaurus Rex
    Join Date
    Jun 2008
    Location
    Redmond, WA
    Posts
    659
    Thanks
    31
    Thanked 100 Times in 94 Posts
    Quote Originally Posted by oesxyl View Post
    1. I like it
    2. not a problem but have some warnings:
    http://jigsaw.w3.org/css-validator/v...xt-arts.com%2F
    3. that is a problem:
    http://www.websiteoptimization.com/s...r.xt-arts.com/
    at least jpg used for layout can have smaller size and load faster.

    regards
    Yeah for the sake of the layout I had to eat up some CSS warnings - which I'm not too much of a fan of because I've got Web-OCD.

    The speed tests are an issue I've been concerned with. It loads pretty fast for me but I do think I should look into some optimization.

    Thanks for the feedback!
    Unless otherwise stated, any code posted is most likely untested and may contain syntax errors.
    My posts, comments, code, and suggestions reflect only my personal views.
    Web Portfolio and Code Snippets: http://shanechism.com

  • #8
    The Apostate Apostropartheid's Avatar
    Join Date
    Oct 2007
    Posts
    3,215
    Thanks
    16
    Thanked 265 Times in 263 Posts
    After swearing a bit because you have those irritating little CSS imports, I noticed that you've forgotten the golden rule of typeface picking:
    Always define backup and generic fonts.
    Quote Originally Posted by global.css, line 12
    Code:
    Font-Family: Arial;
    would be safer defined as
    Code:
    font-family: arial, helvetica, sans-serif;

  • Users who have thanked Apostropartheid for this post:

    ShaneC (07-24-2008)

  • #9
    Codeasaurus Rex
    Join Date
    Jun 2008
    Location
    Redmond, WA
    Posts
    659
    Thanks
    31
    Thanked 100 Times in 94 Posts
    Quote Originally Posted by CyanLight View Post
    After swearing a bit because you have those irritating little CSS imports, I noticed that you've forgotten the golden rule of typeface picking:
    Always define backup and generic fonts.

    would be safer defined as
    Code:
    font-family: arial, helvetica, sans-serif;
    That's so intresting you said that because I was literally just thinking to myself: "Hey. What if them there folks vistin' my web thang don't have them there fonts in there computin' device?"

    I'll implement straight away. Thanks!
    Unless otherwise stated, any code posted is most likely untested and may contain syntax errors.
    My posts, comments, code, and suggestions reflect only my personal views.
    Web Portfolio and Code Snippets: http://shanechism.com

  • #10
    Supreme Master coder! _Aerospace_Eng_'s Avatar
    Join Date
    Dec 2004
    Location
    In a place far, far away...
    Posts
    19,291
    Thanks
    2
    Thanked 1,043 Times in 1,019 Posts
    Looks good. I don't know if the support our cause button fits in with the rest of the site. Its shiny and almost everything else isn't.

    I think the site is too wide but you already knew this. It doesn't scale to smaller resolutions. Depending on your target audience, possibly older republicans (?) it may not suit their resolutions. The UCF part on the header needs some tweaking, I don't know what but it looks a little too plain.

    Are you going to get a real domain or is it going to stay where its at?
    ||||If you are getting paid to do a job, don't ask for help on it!||||

  • Users who have thanked _Aerospace_Eng_ for this post:

    ShaneC (07-24-2008)

  • #11
    Supreme Master coder! abduraooft's Avatar
    Join Date
    Mar 2007
    Location
    N/A
    Posts
    14,858
    Thanks
    160
    Thanked 2,223 Times in 2,210 Posts
    Blog Entries
    1
    The page-content div is shifted about 100px right in IE6.
    The Dream is not what you see in sleep; Dream is the thing which doesn't let you sleep. --(Dr. APJ. Abdul Kalam)

  • Users who have thanked abduraooft for this post:

    ShaneC (07-24-2008)

  • #12
    Codeasaurus Rex
    Join Date
    Jun 2008
    Location
    Redmond, WA
    Posts
    659
    Thanks
    31
    Thanked 100 Times in 94 Posts
    Quote Originally Posted by _Aerospace_Eng_ View Post
    Looks good. I don't know if the support our cause button fits in with the rest of the site. Its shiny and almost everything else isn't.

    I think the site is too wide but you already knew this. It doesn't scale to smaller resolutions. Depending on your target audience, possibly older republicans (?) it may not suit their resolutions. The UCF part on the header needs some tweaking, I don't know what but it looks a little too plain.

    Are you going to get a real domain or is it going to stay where its at?
    I can see what you're saying on the Support our Cause button, I'll play with some alternative designs for that; and for the UCF part I was just thinking that actually, I did the header before any of the rest of the site and it doesn't really jive. I'll fool with that as well.

    As for the resolution yeah, it seems to work on 1064x768 which I'm cool with for now but future layouts I'll definetly change - thanks for all the advice on that it was extremely helpful.

    The domain will be moved when the site is out of beta, the organization I'm making this for has their own domain so it will transplant. I've coded the website to work in any directory or subdirectory it's in (when you look at the source and see the full address that's actually dynamically set when it's recalled from the DB).

    Thanks very much for your input! Extremely helpful, as always.

    Quote Originally Posted by abduraooft View Post
    The page-content div is shifted about 100px right in IE6.
    I just saw this when I was viewing in different IE browsers. Aerospace posted a great IE viewer for me last night but prior to that (and during the coding process) I didn't have access to below IE7. I'm really not sure what's causing that (or how to fix it for that matter) but I'm looking into it.

    Thanks for the heads up!
    Unless otherwise stated, any code posted is most likely untested and may contain syntax errors.
    My posts, comments, code, and suggestions reflect only my personal views.
    Web Portfolio and Code Snippets: http://shanechism.com

  • #13
    Supreme Master coder! _Aerospace_Eng_'s Avatar
    Join Date
    Dec 2004
    Location
    In a place far, far away...
    Posts
    19,291
    Thanks
    2
    Thanked 1,043 Times in 1,019 Posts
    Quote Originally Posted by ShaneC View Post
    I just saw this when I was viewing in different IE browsers. Aerospace posted a great IE viewer for me last night but prior to that (and during the coding process) I didn't have access to below IE7. I'm really not sure what's causing that (or how to fix it for that matter) but I'm looking into it.

    Thanks for the heads up!
    Add display:inline; to the CSS for .page-content

    Its an IE double margin bug. Look it up to understand it better.
    ||||If you are getting paid to do a job, don't ask for help on it!||||

  • #14
    Codeasaurus Rex
    Join Date
    Jun 2008
    Location
    Redmond, WA
    Posts
    659
    Thanks
    31
    Thanked 100 Times in 94 Posts
    Quote Originally Posted by _Aerospace_Eng_ View Post
    Add display:inline; to the CSS for .page-content

    Its an IE double margin bug. Look it up to understand it better.
    Bingo. Many thanks once more
    Unless otherwise stated, any code posted is most likely untested and may contain syntax errors.
    My posts, comments, code, and suggestions reflect only my personal views.
    Web Portfolio and Code Snippets: http://shanechism.com

  • #15
    The Apostate Apostropartheid's Avatar
    Join Date
    Oct 2007
    Posts
    3,215
    Thanks
    16
    Thanked 265 Times in 263 Posts
    Quote Originally Posted by ShaneC View Post
    As for the resolution yeah, it seems to work on 1064x768 which I'm cool with for now but future layouts I'll definetly change - thanks for all the advice on that it was extremely helpful.
    A List Apart may be of service here.


  •  

    Posting Permissions

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