Review my latest project

Hi All!
Can you please review my latest asp project, Room Booking System. Room Booking System is a flexible software package for controlling classroom utilisation for schools and colleges. You can view a demo of the system at:

http://www.roombookingsystem.co.uk/website/demo.asp

Any Comments/Improvements you can think of are much appreciated!

-- Moved to Site Reviews forum

Spelling Errors:

1) Organisation is spelled Organization

2) Efficicent is spelled Efficient

3) matience is spelled maintenance

Error when clicking a date in the "Quick Booking" calendar in the admin section.


You aren't validating your querystring data.
I hacked changed your querystring values and received the following error.
I actually received different types of errors depending on the querystring value I changed.

I didn't have time to try any javascript hacking though, so you may also have issues there.

--comments:
1. the screen shots are too small, make them links to the full screen shots.
2. spelling errors in the main page. makes it look not professional.
3. the Copyright statement is better be written in smaller font and further down the page.
4. email in the contact page should be link.
5. as Memnoch said, clicking the dates in the calendar generates error. should not happen even in demo version - simply remove the calendar if it's not active in demo.
6. you better separate the username used to log in (e.g. "demo") from the real user name i.e. add another field to the database. this way it would feel more nice to use the system.
7. the html is not valid: in the main page you miss this:

and in other pages you have js code appear in the beginning of the html - move the js inside the head section of the page.


anyhow the general design is nice and simple, the website works fine with Firefox.

Not in the UK (we spell it correctly )

All other comments are agreed with

I like it. It looks simple and therefore it's fine.
On your admin side, you have errors when trying to edit classes.
All this in addition to what's been said before.

Thanks For All Your Comments!

I'll get onto correcting the spelling mistakes right away.

To answer Shadow Wizard's 1th Point, I really like the idea of making the screen shots full size when you click on them, i'll include this on the website asap.

To answer Shadow Wizard's 6th Point, there is a separate field in the database which contains the users forename and surname (I think this is what you mean)

Again, to answer his 7th point, it would be a right pain to edit every page to include the validating so i'm not sure if I should do this. Does anyone know the advantages of doing this?

It is unusual that the calendar and the classes page (in admin) do not work as they seem fine in 'live' versions of the system. I'll see if copying the working files over the ones with errors works.

Thanks again for all your comments/improvements!

cool, so you better use those fields in the welcome message.


I don't understand - what you mean by "validating"?? I said the html is wrong.... it means it won't pass any validator, and can mess up the whole page in standard browsers. advantage of having correct html? hmm... I can see some advantages.

yes...it's called professionalism.....and if you build it that way...they will come

in other words....you have to build a site around the users browser...w/o being html compliant...it's doomed

Not allowed to rep ya - but well put

proxy repped for you!