1. Advertising
    y u no do it?

    Advertising (learn more)

    Advertise virtually anything here, with CPM banner ads, CPM email ads and CPC contextual links. You can target relevant areas of the site and show ads based on geographical location of the user if you wish.

    Starts at just $1 per CPM or $0.10 per CPC.

Please critic my site!

Discussion in 'HTML & Website Design' started by talkmeow, Jul 13, 2009.

  1. #1
    This is my first website:). Any tips or suggestions would be greatly appreciated! Thanks!

    www.greatdealguru.com
     
    talkmeow, Jul 13, 2009 IP
  2. ngdesign

    ngdesign Peon

    Messages:
    10
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #2
    i like the layout, clean and unconfusing
     
    ngdesign, Jul 13, 2009 IP
  3. kosyer

    kosyer Peon

    Messages:
    10
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #3
    not bad u need add some other things like post comments etc with captcha
     
    kosyer, Jul 13, 2009 IP
  4. webtemppro

    webtemppro Guest

    Messages:
    12
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #4
    this theme has been overused, but it is elegant
     
    webtemppro, Jul 13, 2009 IP
  5. talkmeow

    talkmeow Greenhorn

    Messages:
    90
    Likes Received:
    3
    Best Answers:
    0
    Trophy Points:
    23
    #5
    Wow thanks guys, I was expecting the worst haha. :)
    I made a few tweaks to it, and I still need to get captcha. Any other feedback would be great!
     
    talkmeow, Jul 13, 2009 IP
  6. lucysam123

    lucysam123 Guest

    Messages:
    30
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #6
    nice site! clear and the design is kept simple :)
     
    lucysam123, Jul 14, 2009 IP
  7. garrettheel

    garrettheel Peon

    Messages:
    341
    Likes Received:
    11
    Best Answers:
    0
    Trophy Points:
    0
    #7
    I've seen that theme way too many times to make the site look interesting..

    If you're not comfortable starting from scratch and you like that theme, you need to tweak it a bit to be more unique and relevant to your site. For example have an actual logo at the top (image) instead of just text, and changing the background wouldn't be a bad idea
     
    garrettheel, Jul 14, 2009 IP
  8. Payton Designs

    Payton Designs Peon

    Messages:
    45
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #8
    Its a nice design, but definitely not unique at all. Sometimes being unique is the key.
     
    Payton Designs, Jul 14, 2009 IP
  9. kittyluver

    kittyluver Notable Member

    Messages:
    6,812
    Likes Received:
    222
    Best Answers:
    0
    Trophy Points:
    263
    #9
    The design is really awesome. Try to build unique content.
     
    kittyluver, Jul 14, 2009 IP
  10. talkmeow

    talkmeow Greenhorn

    Messages:
    90
    Likes Received:
    3
    Best Answers:
    0
    Trophy Points:
    23
    #10

    That is a good idea, I'm going to look for a nice image to put as my logo.

    And I will look into other ways of making my site unique. Thanks for the feedback everybody :)
     
    talkmeow, Jul 14, 2009 IP
  11. mcreation

    mcreation Banned

    Messages:
    44
    Likes Received:
    1
    Best Answers:
    0
    Trophy Points:
    0
    #11
    Not So bad design. Its Good
     
    mcreation, Jul 14, 2009 IP
  12. dgroom

    dgroom Guest

    Best Answers:
    0
    #12
    It is looking ok, there is 1 simple error in the XHTML validation:

    Line 57, Column 133: Attribute "border" exists, but can not be used for this element.
     
    dgroom, Jul 15, 2009 IP
  13. touster

    touster Peon

    Messages:
    64
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #13
    Nice, clean and, simple. Overall a good layout.
     
    touster, Jul 15, 2009 IP
  14. Cheap SEO Services

    Cheap SEO Services <------DoFollow Backlinks

    Messages:
    16,664
    Likes Received:
    1,318
    Best Answers:
    0
    Trophy Points:
    0
    #14
    It could use a little more yellow up top.
     
    Cheap SEO Services, Jul 15, 2009 IP
  15. Martinoes

    Martinoes Peon

    Messages:
    110
    Likes Received:
    2
    Best Answers:
    0
    Trophy Points:
    0
    #15
    'Subscribe to' buttons are under google ads panel when you move mouse over 'RSS feed'.
     
    Martinoes, Jul 15, 2009 IP
  16. levston

    levston Peon

    Messages:
    14
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    0
    #16
    Terrible banner
     
    levston, Jul 15, 2009 IP
  17. webhostdeals

    webhostdeals Active Member

    Messages:
    126
    Likes Received:
    1
    Best Answers:
    0
    Trophy Points:
    53
    #17
    If i have to rate it from 1-10 id rate to 5
     
    webhostdeals, Jul 15, 2009 IP
  18. Destinyx3

    Destinyx3 Peon

    Messages:
    430
    Likes Received:
    5
    Best Answers:
    0
    Trophy Points:
    0
    #18
    I also have seen that theme way too much. Add some uniqueness to it and you will stand out with the theme.

    Other than that, the blog is nice overall, so there's nothing to worry about.
     
    Destinyx3, Jul 15, 2009 IP
  19. codenamerana

    codenamerana Member

    Messages:
    9
    Likes Received:
    0
    Best Answers:
    0
    Trophy Points:
    36
    #19
    Design Sense : 6/10
    Content Sense : 4/10
    :D
     
    codenamerana, Jul 16, 2009 IP
  20. deathshadow

    deathshadow Acclaimed Member

    Messages:
    9,732
    Likes Received:
    1,998
    Best Answers:
    253
    Trophy Points:
    515
    #20
    Fixed metric fonts are an accessability /FAIL/, fixed width layout is not netbook or other mobile device friendly, it's not entirely clear where each article starts/ends (needs more padding), and the text in the search bar is illegible due to being too low a contrast to the background.

    It appears to be designed in XHTML 1.1 which there is no point to even be trying to do given IE's complete lack of support for it - it brings nothing to the table in terms of cross-browser deployment that 1.0 STRICT doesn't, and it can introduce rendering errors like those I'm seeing in IE6 - though those errors could also be blamed on the placement of the semi-pointless comments.

    I'm seeing clearing div's, unnecessary DIV's and Classes all over the place, and a whole host of other issues that really should be cleaned up.

    The layout seems to work (for now) in all modern browsers EXCEPT IE, where on my system it appears to be slightly broken on a few things in IE8 - and it's next to useless in IE6 and 7 from the disappearing content and double-render bugs tripped by the comment placement.

    The commenting is the pinnacle of wasteful redundant bad comments too...

    <!-- wrap START -->
    <div id="wrap">
    
    <!-- container START -->
    <div id="container"  >
    
    <!-- header START -->
    <div id="header">
    Code (markup):
    Really? Opening a tag with an ID is the start of a section with the same name? Who'd have thought!?!

    Comments between elements if floats are involved can make both IE6 and IE7 completely wierd out. Hence, it's usually a good idea to not bother with the pointless redundant comments at the start like you have above, and when it comes to closing the elements instead of this:

    </div>
    <!-- header END -->
    Code (markup):
    use this
    
    <!-- #header --></div>
    Code (markup):
    We know it's the end - that's what </div> means. Moving the comment before the close prevents the comment from ending up in-between elements and therein preventing the IE comment render bugs.

    As to the rest, it's not bad... For all the above comments I've seen far worse as you've not got an unreasonable layout, the color choices (except for the dark backround on the search) are quite nice, and the use of visual eye-candy is done tastefully and does not overpower the important part of the page - the content.
     
    deathshadow, Jul 16, 2009 IP