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 5 of 5
  1. #1
    Regular Coder bacterozoid's Avatar
    Join Date
    Jun 2002
    Location
    USA
    Posts
    490
    Thanks
    24
    Thanked 35 Times in 35 Posts

    Review of the site...

    Figure I might as well opt for some comments...whether or not people give them is up to that individual...

    http://www.beautifulorigin.com

    --I do that site for a local band here...was wondering what people with experience think of it. I know people who know nothing about web-building or design like it, but I was curious as to some more professional comments.

  • #2
    New Coder
    Join Date
    Jul 2002
    Location
    London, UK
    Posts
    92
    Thanks
    0
    Thanked 0 Times in 0 Posts
    It looks good enough. But the code is very untidy and does not validate (to HTML 4.01 transitional, since you specified no doctype). Validation results: http://validator.w3.org/check?uri=ht...C+worldwide%29
    skinme.net / v2.php (design nearly completed)

  • #3
    Senior Coder
    Join Date
    Oct 2003
    Location
    Australia
    Posts
    1,963
    Thanks
    0
    Thanked 0 Times in 0 Posts
    * Your CSS for a:link,visited,etc. doesn't work in Mozilla. They show as default, which is ugly and difficult to read over the dark BG.

    *The splash page detracts from the site. It is not there for any purpose (does not provide site specs, browser/plug - in requirments) so it is only excess bandwidth.

    * The hard edges of the photos in the header are a little off putting, especially the one between the T and I in Beautiful. As much as I hate bodgey montage's, I think you may need to montage these images.

    * For my eye, there isn't enough contrast between the three background colours you've used. I would make the area surrounding the content lighter.

    *The >> before the navigation is really tacky and dated. If you want a bullet, do it in CSS with display:block;padding-left:10px;background-img:url (image.gif); in the CSS for the links (<a>)

    *Your code is a bit of a mess
    You've got CSS that doesn't work, all sorts of deprecated tags (<font> is a BIG no-no for new pages USE CSS!)
    tables all over the shop and calls to classes that don't exist

    <p class="papy" align="justify">
    You asked for it

    **edit: AND you don't have a Doctype or Character set declared...

    I take no responsibility for the above nonsense.


    Left Justified

  • #4
    Regular Coder bacterozoid's Avatar
    Join Date
    Jun 2002
    Location
    USA
    Posts
    490
    Thanks
    24
    Thanked 35 Times in 35 Posts
    Heh, thanks. The site used to be old and completely (for the mostpart) done in CSS, which is where classes come in that don't exist...I've updated stuff. I do suppose I should go through and redo it all I guess...depending on if it's worth my time or not.

    I don't know all I should to be a pro webmaster; it's more or less a hobby I guess...thanks for the tips...basically I'm out of date is what it seems like.

  • #5
    Senior Coder
    Join Date
    Jun 2002
    Location
    The Netherlands, Baarn, Ut.
    Posts
    4,252
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Invest time to save time

    I do suppose I should go through and redo it all I guess...depending on if it's worth my time or not.
    You may have to spend quite a bit of time initially to convert it all (you could do it in increments, taking on the biggest eye sores first and refining it step by step), but you should experience that in the end, maintenance on leaner, cleaner pages will save you a lot of time.
    Regards,
    Ronald.
    ronaldvanderwijden.com


  •  

    Posting Permissions

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