Initial code review of current code base

Thinking about it - it might also be nice to have these help URL's defined in a single place...  If they were defined as constants then it could be passed to the BasePage constructor as one of it's init params - that would be quite nice and avoid another virtual