Time to get reviewed

[cpeters]

Good aftrernoon all,
I have finally got just about all of the elements up and working. So It is time for the established peers of this realm to look, see, pick apart and tell me what they think!:)

The domain is http://www.padraighs-alehouse.com Let loose the dogs of the net! :)

[bdoc]

Looks good.... Nice simple, easy to follow design. I can dig the color scheme as well as the setup.

My suggestion....

I'm not a fan of the "No Right Click" script, chances are that if someone really wants to have a look at your code or take an image they will most certainly find a way.

Otherwise, you've got a nice looking site!

[(jj)]

Nice K.I.S.S. design and easy to navigate. Only have a couple of comments :)

You should open the off-site links, like your PENNSIC link on the main page, in a new browser window. Helps to keep your site open so you do not lose the visitor.

You have one bad link. On your "Fees" page, the link to "Needs" is incomplete, it points to /needs not /needs.php

I also agree with bdoc on the use of the no right click script. What are you trying to protect? If someone truly wants your code or an image, that will not even slow them down.

[rafeflagg]

I concur with the prior posts.

My only add is that on Http://www.padraighs-alehouse.com/background.php you have a sentence starting "Padraighs Alehouse is a drean your humble..." I believe you meant to say "a dream your humble..."

Other than that, it looks good. :)

rf

[Grymwulf]

Yes I ran into the no right click problem my self, I am not a big fan of it, and if I chose to look at the source ... wow I can steal anything I want now, including your no right click script ...

// (C) 2000 www.CodeLifter.com
// http://www.codelifter.com
// Free for all users, but leave in this header

// Set the message for the alert box
am = "Copyright 2003 Padraighs-alehouse.com. If you want to use anything here contact padrhaig@padraighs-alehouse.com";

// do not edit below this line
// ===========================
bV = parseInt(navigator.appVersion)
bNS = navigator.appName=="Netscape"
bIE = navigator.appName=="Microsoft Internet Explorer"

function nrc(e) {
if (bNS && e.which > 1){
alert(am)
return false
} else if (bIE && (event.button >1)) {
alert(am)
return false;
}
}

document.onmousedown = nrc;
if (document.layers) window.captureEvents(Event.MOUSEDOWN);
if (bNS && bV<5) window.onmousedown = nrc;

I like the scheme, my only issue is layout ... what's the point of the celtic design on the left if it does not contain any buttons to click???? Plus the length of the front page obscures half of the links at the bottom, so unless you scroll down you don't catch all of the links.

Broken Links:
http://www.padraighs-alehouse.com/info.php ("Who We Are")
http://www.padraighs-alehouse.com/needs ("Camp Needs")

[cpeters]

Sorry so long to reply to all and THANK YOU for dome grat critiques! The code lifter makes it harder for those who "aren't in the know" to try to get my code. Was burned VERY badly on a commercial project several years back. The text is indented in away from the celtic motif to make it easier to read and the motif just looks darn cool! Once again thanks to all!