Moves footer into nav to fix main body size issues

[Keeeeeeeen]

Moves footer into nav to fix main body size issues, which could potentially cause header to cover content

[Keeeeeeeen]

Not ready to merge: Just realised I removed the feedback prompt

[Keeeeeeeen]

Okay now the feedback prompts there, future note, maybe we could cut down on how much we're putting in the footer? it's getting quite wordy and is starting to look a little bloated on mobile imo

[jenkins-ystv]

Deployed a preview of this PR to https://pr-141-internal.dev.ystv.co.uk

[archessmn]

Some thoughts:
I do wonder how necessary the footer actually is with the current usage:

• Version is only really something that people doing development need to know.
• Built and maintained by the computing team isn't something that really needs to be advertised to the people who are using it, seeing as they're almost all exclusively YSTV members who are aware comp team made it.
• Feedback is the only thing that really needs to be available to the average user, but even then that could just be put in the user menu dropdown.

As you said, it just seems too much on mobile. I think quite a bit of the navigation / layout needs some tweaking as aside from the profile dropdown, I don't think it's really been changed since the very early builds.

[Keeeeeeeen]

Definitely agree, thinking I might take this PR back to the drawing board a bit and turn it into a bit more of a general 🫡 navigational overhaul

[Keeeeeeeen]

Actually thinking on it a little more, will probably fold this in with the homepage as most of the info in the footer could probably be moved there too

[archessmn]

That makes sense, one thing we really need is a proper nav menu now that we're adding more pages at the root, I may have a look at doing that tomorrow unless you're wanting to do that as part of this?

[Keeeeeeeen]

Will probably look into it as part of this