Bradford Arms Hotel

R8515198 · #1 Nov 8th, 2007, 15:08

Good Afternoon All,

As I have learnt more I have done a lot of changes to this site.

The site orginally was my first attempt and envolved alot of scrolling, I have made it lot smaller.

I would like any improvements comments.

Generally any comments good/bad will be helpful, as we all have a learning curve and I am still on mine.

(!the logo in the bottom right friendsworld isnt up and working at the mo!)

Cheers
Dan

R8515198 · #2 Nov 8th, 2007, 15:10

Just had a look at the Validate HTML, didnt realise it had errors will correct tonight.

karinne · #3 Nov 8th, 2007, 15:34

It's not bad but I have to be honest here ... I'm not really sure what it looked like before so I can' comment on the improvements but ... what I'm seeing now I don't really like. I feel like it's a site that was done in the 90s

Some inspiration might help - restaurant sites from css mania

alexgeek · #4 Nov 8th, 2007, 15:42

The green is hideous to be honest.

This page,

HTML: Select all

We offer two non smoking areas in which to enjoy our   excellent home cooked cuisine.&nbsp; <br>
<br>
The conservatory is spacious and offers a   relaxed atmosphere.&nbsp; <br>
<br>
The restaurant is more formal and ideal for private   parties.<br>
<br>
We use only the finest local produce to ensure that we   serve first class meals.&nbsp; <br>
<br>
Come and enjoy a light lunch, 3 course evening meal or   a traditional Sunday lunch.

This would be more semantic in an unordered list (<ul>).

R8515198 · #5 Nov 8th, 2007, 15:54

Quick replys,

honesty is the best policy.

Will look into changing it

karinne · #6 Nov 8th, 2007, 16:20

Also ... you don't need the JS for the nav. Simple CSS could achieve the same result.

HTML: Select all

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
 "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" lang="en-CA" xml:lang="EN-CA">

<head>
    <title>Some title</title>
    <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
    
    <style type="text/css">
    #topmenu {
        padding-top: 10px;
        margin-bottom: 20px;
    }

    #topmenu ul {
        list-style: none;
        display: inline;
    }

    #topmenu ul li {
        float: left;
        position: relative;
        margin-right: 5px;
    }
    
    #topmenu ul li a {
        text-decoration: none;
        display: block;
        font-size: 17px;
        letter-spacing: 1px;
        font-weight: bold;
        color: #fff;
        padding: 0 20px;
        background-color: #060;
    }

    #topmenu ul li a:hover {
        background-color: #900;
        
    }
    </style>    
</head>

<body>

<div id="topmenu">
    <ul>
        <li><a href="/">menu</a></li>
        <li><a href="/">menu</a></li>
        <li><a href="/">menu</a></li>
        <li><a href="/">menu</a></li>
        <li><a href="/">menu</a></li>
    </ul>
</div>

</body>
</html>

R8515198 · #7 Nov 8th, 2007, 16:41

Quote:

Originally Posted by karinne

It's not bad but I have to be honest here ... I'm not really sure what it looked like before so I can' comment on the improvements but ...

Here is a archive link to the previous site

http://www.bradfordarmshotel.com/archive/index.html

Marc · #8 Nov 8th, 2007, 18:16

The header image is nice, it really shows off the Hotel's outside. As Karinne said, you don't need Javascript or Images for a navigation like yours.

The images around the site are all really bad quality, especially the picture of the pool table.

You also aren't using <p> tags.....

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Bradford Arms Revamp	R8515198	Free Web Site Critique	7	Jun 30th, 2007 12:18

		Webforumz.com > Main Forums > Design Your Website > Free Web Site Critique
Bradford Arms Hotel