The Building Experience

Hi Guys,

I've just finished creating my first website and am looking for some honest feedback. I posted the link in the welcome forum but was told this is the place to get the best feedback.

Please be gentle with me, i'm only a novice.

Nice site mate, well done for going for css rather than tables for the layout.

Ditch the background. Just make it white. Maybe have a 1px black/purple border around the main page container. Just have the background as white.

That repeated background spoils it for me.

Nice site especially if you are as you say a novice.

Well done.

PS check validation, I don't have time. If it validates well done! if not just fix the errors

I don't mind the background actually, although it says .co.uk but the page is really .com (I know the .co.uk. works, but it redirects to .com, so I'm not sure why you have .co.uk in the background?).

The site looks good. That's a different sort of a business there, I haven't heard of something like that before, kind of a Do-It-Yourself-With-Us deal (I want credit if you use that )

Yes, nice!

Very nice. I agree with 1840dsgn. I would prefer you not have the page background. It's distracting away form the gorgeous site.
I'd love to see the softest lilac/lavender, or gray in the sky area. I think it could use a border around the main div too!

I haven't got time for a more thorough look but overall on a quick glance...very nice!

Don't like the background. I agree with 1840dsgn on the 1px border around it..

If this is seriously your first ever website you've got some talent!

I've just had a look through your css, very in detail and nice work.

I edited it myself, first adding a border made a great diference and I really like it so I'd suggest that.

Secondly, I took awya the bg image and also prefer it, all the image does is take viewers eyes away from the main content.

Really good work though. Well done.

You have got 5 validation errors but they're easy to fix.

Just use & instead of the '&' character.

When coding nested lists, extra ul tags must be within an li tag...
for example

Code: Select all

<ul>
<li>Shopping</li>
<li>
    <ul>
         <li>Bread</li>
         <li>Milk</li>
    </ul>
</li>
<li>Gardening</li>
</ul>

Notice the nested <ul> is within an <li> from the first level list. Hope that makes sense.

You've also got an error for not closing one of your <li> tags, however solve the nested list and this might well disappear.

You've used a couple of <font> tags, you no this is bad practice cos you've done most of it in CSS, so get rid of these and use CSS instead, these are giving you a couple of errors.

Sort those errors out and well done

After the helpful feedback i recieved from everyone i have changed the background on my website. Would love to know what everyone thinks.

http://thebuildingexperience.com

I know it's quite bright, but i'm trying to create a brand with purple as the company colour.

Cheers

Jake

Love it!!! Brave use of color and it works!!! Congrats