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.
Page 1 of 2 12 LastLast
Results 1 to 15 of 18

Thread: Need Critique

  1. #1
    New Coder
    Join Date
    Aug 2007
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Need Critique

    Hello,

    Can you rate my website, http://www.dadari.com/ ?

    Any and all critics are appreciated.

    Thanks!

  • #2
    Regular Coder croatiankid's Avatar
    Join Date
    Jan 2006
    Posts
    665
    Thanks
    1
    Thanked 12 Times in 12 Posts
    Not enough color, your footer's right border seems to be off a little, the black boxes aren't too pretty, and the header looks too plain, and I think you should put something else instead of Dadari.Com.

  • #3
    Senior Coder rafiki's Avatar
    Join Date
    Aug 2006
    Location
    Floating around somewhere...
    Posts
    2,039
    Thanks
    19
    Thanked 42 Times in 42 Posts
    didnt quite pass the CSS validator either
    Code:
    Sorry! We found the following errors
    URI : http://www.dadari.com/
    56 		Value Error : font-size only 0 can be a length. You must put an unit after your number : 11
    57 		Value Error : font-size only 0 can be a length. You must put an unit after your number : 11
    82 		Value Error : font-size only 0 can be a length. You must put an unit after your number : 11
    83 		Value Error : font-size only 0 can be a length. You must put an unit after your number : 11
    also
    Code:
    This page is not Valid XHTML 1.0 Transitional!
    Result: 	Failed validation, 73 Errors
    should get back to work to make the errors disappear

  • #4
    New Coder
    Join Date
    Aug 2007
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts
    croatiankid - Can you give me some alternatives?

    I have been looking into changing those but so far, no ideas. O_O


    rafiki - Thanks for pointing out the CSS Errors.

    It used to be Valid XHTML Transitional, but ever since i got cutenews, more errors have popped out, and Im too lazy to fix that many errors.

  • #5
    New Coder
    Join Date
    Aug 2007
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by rafiki View Post
    didnt quite pass the CSS validator either
    Code:
    Sorry! We found the following errors
    URI : http://www.dadari.com/
    56 		Value Error : font-size only 0 can be a length. You must put an unit after your number : 11
    57 		Value Error : font-size only 0 can be a length. You must put an unit after your number : 11
    82 		Value Error : font-size only 0 can be a length. You must put an unit after your number : 11
    83 		Value Error : font-size only 0 can be a length. You must put an unit after your number : 11
    also
    Code:
    This page is not Valid XHTML 1.0 Transitional!
    Result: 	Failed validation, 73 Errors
    should get back to work to make the errors disappear
    Can you help me with validating the CSS?
    I really don't get those errors I keep getting.

  • #6
    Regular Coder croatiankid's Avatar
    Join Date
    Jan 2006
    Posts
    665
    Thanks
    1
    Thanked 12 Times in 12 Posts
    The problem with your CSS is that you haven't specified a unit for your strings. It even says it. You must define a unit, be it 11px, em, %, in, w/e.

  • #7
    New Coder
    Join Date
    Aug 2007
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by croatiankid View Post
    The problem with your CSS is that you haven't specified a unit for your strings. It even says it. You must define a unit, be it 11px, em, %, in, w/e.
    margin:0;
    padding:0;

    Such as that space?

  • #8
    Regular Coder Graft-Creative's Avatar
    Join Date
    Aug 2004
    Location
    Web Designer - North East Lancashire U.K.
    Posts
    842
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by FinishUpper View Post
    margin:0;
    padding:0;

    Such as that space?
    ...a unit for your font size, i.e.
    Code:
    font-size: 11px;
    Gary

  • #9
    New Coder
    Join Date
    Aug 2007
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by Graft-Creative View Post
    ...a unit for your font size, i.e.
    Code:
    font-size: 11px;
    Gary
    Yes, I know that....

    its just not working.

  • #10
    Senior Coder rafiki's Avatar
    Join Date
    Aug 2006
    Location
    Floating around somewhere...
    Posts
    2,039
    Thanks
    19
    Thanked 42 Times in 42 Posts
    love the comment at the bottom of your css lol
    /* This stylesheet was made by Dadari.Com. Please do not steal. */
    lol

  • #11
    Regular Coder
    Join Date
    Jul 2007
    Location
    USA
    Posts
    146
    Thanks
    5
    Thanked 3 Times in 2 Posts
    Well from my view it's a pretty sad site. You should get a lot more content up. I think I went through your entire site in about 2 minutes tops. If you're planning on putting more content up soon that would be very helpfull to your site. If not well then I don't know what to say. The layout is decently attractive for a first time layout. The banners a bit... bad no offense. Hopefully you can add some more content and fix those css errors! Good luck.
    Quote Originally Posted by rmedek View Post
    Doctordew, as soon as they come out with the opposite of an infraction, I am going to give you a million of them. You are my new favorite person on the forum.

  • #12
    New Coder
    Join Date
    Aug 2007
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by doctordew View Post
    Well from my view it's a pretty sad site. You should get a lot more content up. I think I went through your entire site in about 2 minutes tops. If you're planning on putting more content up soon that would be very helpfull to your site. If not well then I don't know what to say. The layout is decently attractive for a first time layout. The banners a bit... bad no offense. Hopefully you can add some more content and fix those css errors! Good luck.
    Yes, I know... Im trying to fix the css errors.

  • #13
    Senior Coder ahallicks's Avatar
    Join Date
    May 2006
    Location
    Lancaster, UK
    Posts
    1,134
    Thanks
    1
    Thanked 57 Times in 55 Posts
    CSS errors seem to be fixed now. As for the XHTML... CuteNews allows you to edit the template it uses to display the news... this is all you'll need to do to get rid of most of the errors on that page. Take out the inline styles and change the table into a set of divs that you can easily style with CSS. Remove all of that font crap too, and set the fonts for the divs you create in the news template and everything should be fine. I've found that the beauty of CuteNews is that you can edit the output template as much as you want. I use it on my sites and they validate no problems.
    "write it for FireFox then hack it for IE."
    Quote Originally Posted by Mhtml View Post
    Domains are like women - all the good ones are taken unless you want one from some foreign country.
    Reputation is your friend

    Development & SEO Tools

  • #14
    New Coder
    Join Date
    Jul 2007
    Location
    USA
    Posts
    83
    Thanks
    6
    Thanked 7 Times in 7 Posts
    You may want to post exactly what your website's purpose is on the homepage.

    Usability expert Jakob Nielsen gives 10 things to do on a homepage:

    http://www.useit.com/alertbox/20020512.html

    You are missing some critical information which hurts usability.

  • #15
    New Coder
    Join Date
    Aug 2007
    Posts
    11
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by ahallicks View Post
    CSS errors seem to be fixed now. As for the XHTML... CuteNews allows you to edit the template it uses to display the news... this is all you'll need to do to get rid of most of the errors on that page. Take out the inline styles and change the table into a set of divs that you can easily style with CSS. Remove all of that font crap too, and set the fonts for the divs you create in the news template and everything should be fine. I've found that the beauty of CuteNews is that you can edit the output template as much as you want. I use it on my sites and they validate no problems.
    Thanks for the cutenews info...

    But the css errors haven't gone away. o_O


  •  
    Page 1 of 2 12 LastLast

    Posting Permissions

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