Fix up tests to reflect new error layout
There's more to do to clean up the app but we can launch with this.
Made form layout a bit more consistent, tried to clean up error handling
This will now display a nice message to our users.
Don't want to use inline styles, it blats any other styling.
There was one field which had a contradictory label, for which I removed the required-ness.
This should allow developers/designers to keep the working production credentials (needed to view almost every page) in their local checkout, without it being committed and published.
From the point where we moved to Rails, the forms weren't working outside of automated tests. This is because we didn't have the CSRF token set inside each form, but - unless you were paying close attention - it looked like it had succeeded because the CSRF error triggered an OAuth loop which ended in the user being shown the homepage, rather than any kind of error. By using the form_tag helper, we get the CSRF token (authenticity_token) and UTF8 detection magic to boot.
This static message was always being included, no need to be in a separate file.
We now use application base layout instead.