Cleanup docs Dockerfile, so it matches best practices.

- Cleanup spacing throughout
- swap `&&` to beginning of lines to improve readability
- better cache busting for the SvenDowideit/mkdocs repo based on git commit id
- take advantage of the `WORKDIR`

Thanks to @tianon for the review so far :smile:.

I made this and realized I had included the same changes as @jfrazelle in #8715, but I think this adds a few more tweaks to make it more readable.
Shouldn't it be, "commit hash of the newest commit from SvenDowideit/mkdocs..."?