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 8 of 8
  1. #1
    New Coder
    Join Date
    Jul 2011
    Location
    Canada
    Posts
    60
    Thanks
    5
    Thanked 0 Times in 0 Posts

    Smile Site for practice, first site (:

    Hi, I posted a topic about a month ago on this site that I'm working on, It's only a site for practice since I have only been coding for about 6-7 months. (:

    Tell me how it looks, how I could improve it, any constructive criticism would help me. (:

    The alert on the homepage is only temporary and I plan to remove it in the future.

    The forum design wasn't made by me and I'm fully aware of that.
    I even left the copyright at the bottom of the forums.
    I'm thinking about making my own style, but I haven't confirmed that I will yet.

    Please take a look and tell me how I've done. (:

    Link: http://www.moderncoders.tk

    Thanks,

    Sam.
    Check out my new forums: http://www.moderncoding.co.nr (:

    Strengths: HTML & CSS.


    Code:
    EpicFriends x = 
    players.getNearest(BestFriend);
    if (x != null) {
    x.interact("High Five" + x.getName());

  • #2
    New Coder
    Join Date
    Jul 2011
    Location
    Canada
    Posts
    60
    Thanks
    5
    Thanked 0 Times in 0 Posts
    50+ views and no reply, is it that bad?
    Check out my new forums: http://www.moderncoding.co.nr (:

    Strengths: HTML & CSS.


    Code:
    EpicFriends x = 
    players.getNearest(BestFriend);
    if (x != null) {
    x.interact("High Five" + x.getName());

  • #3
    Regular Coder
    Join Date
    Mar 2012
    Posts
    168
    Thanks
    5
    Thanked 11 Times in 11 Posts
    There isn't really anything to it... but it definitely works.

  • #4
    New Coder
    Join Date
    Jul 2011
    Location
    Canada
    Posts
    60
    Thanks
    5
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by KULP View Post
    There isn't really anything to it... but it definitely works.
    Atleast you replyed and took the time to give me some feedback, thank you. (:
    Check out my new forums: http://www.moderncoding.co.nr (:

    Strengths: HTML & CSS.


    Code:
    EpicFriends x = 
    players.getNearest(BestFriend);
    if (x != null) {
    x.interact("High Five" + x.getName());

  • #5
    New Coder
    Join Date
    Sep 2010
    Location
    Beaverton, OR
    Posts
    38
    Thanks
    3
    Thanked 2 Times in 2 Posts
    Hey Sam,

    It's okay for still being new. I have some notes I think could help you.

    When using a background image, one shouldn't be able to see where the image ends. It can be tough with an image as bold as the one you've chosen. Perhaps something similar could work, but I'd make sure you couldn't see the edges.

    The section with the content (main-wrap) could be improved with some padding all around. Those abrupt edges can be distracting, and not so pleasing to the eye.

    I'd increase the line-height in the description. This makes it easier to read, and it makes the site look cleaner.

    I'd lose the welcome text near the top. It's distracting, and it's not done much, anymore.

    The design seems a little disjointed. It's important for a design to fit within itself. There seem to be different styles and ideas competing. I'd pick one, and stick with it.

    The image above the welcome text is broken.

    Some sites that have been helpful for me:
    http://colorschemedesigner.com/
    http://browsersize.googlelabs.com/
    http://validator.w3.org/unicorn/

    Good luck, dude. Keep at it!

  • #6
    New Coder
    Join Date
    Jul 2011
    Location
    Canada
    Posts
    60
    Thanks
    5
    Thanked 0 Times in 0 Posts

    Smile

    Quote Originally Posted by chrisrozwod View Post
    Hey Sam,

    It's okay for still being new. I have some notes I think could help you.

    When using a background image, one shouldn't be able to see where the image ends. It can be tough with an image as bold as the one you've chosen. Perhaps something similar could work, but I'd make sure you couldn't see the edges.

    The section with the content (main-wrap) could be improved with some padding all around. Those abrupt edges can be distracting, and not so pleasing to the eye.

    I'd increase the line-height in the description. This makes it easier to read, and it makes the site look cleaner.

    I'd lose the welcome text near the top. It's distracting, and it's not done much, anymore.

    The design seems a little disjointed. It's important for a design to fit within itself. There seem to be different styles and ideas competing. I'd pick one, and stick with it.

    The image above the welcome text is broken.

    Some sites that have been helpful for me:
    http://colorschemedesigner.com/
    http://browsersize.googlelabs.com/
    http://validator.w3.org/unicorn/

    Good luck, dude. Keep at it!
    Thanks so much for your feedback man!

    I will work on everything you said and I will keep at it.

    I really appreciate you taking your time.

    Sam. (:
    Check out my new forums: http://www.moderncoding.co.nr (:

    Strengths: HTML & CSS.


    Code:
    EpicFriends x = 
    players.getNearest(BestFriend);
    if (x != null) {
    x.interact("High Five" + x.getName());

  • #7
    The fat guy next door VIPStephan's Avatar
    Join Date
    Jan 2006
    Location
    Halle (Saale), Germany
    Posts
    8,609
    Thanks
    6
    Thanked 997 Times in 970 Posts
    One suggestion, code wide: Validate your HTML. The errors almost don’t surprise me, considering that you have learned from the W3 Fools.

    Also, don’t use HTML comments in stylesheets or scripts. These are outdated remainders of old practices and not necessary anymore – and they just show that you seem to have learned from outdated (or at least not properly updated) sources.

    And then – text-decoration: blink??? Man, I thought the 90’s are over? Design wise this is considered very amateur, as is the effect on the welcome text. I mean, it would be OK if it would appear one time and stay still then. But the way it is it becomes annoying and distracts from the actual content of the page (if this was a real-life site).

  • #8
    New Coder
    Join Date
    Jul 2011
    Location
    Canada
    Posts
    60
    Thanks
    5
    Thanked 0 Times in 0 Posts

    Smile

    I have posted a new thread with my new design.

    I have taken in all of your recommendations and have edited my page to the best of my abilities, I'm still looking for a better navbar design or a tutorial helped me design one with css3.

    Hope you guys enjoy it and if you have any other recommendations, please post them on the new thread.

    Link: First Site Little Bit of Redesigning


    Thanks,

    Sam.
    Check out my new forums: http://www.moderncoding.co.nr (:

    Strengths: HTML & CSS.


    Code:
    EpicFriends x = 
    players.getNearest(BestFriend);
    if (x != null) {
    x.interact("High Five" + x.getName());


  •  

    Posting Permissions

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