is it worth breaking styles into a separate file?  nit here is that we've got included and inline styles, question is one of taste on one hand, and best practices on the other.  For the latter point we could use a Content-Security-Policy that disables inline scripts.  Though arguably that decreases readability.

I don't feel strongly here, if you think this is aesthetically better, let's merge as is - your call.