Phasing out the inline style attribute

[trichoplax]

Inline styles are not incorrect, and our style guide does not currently recommend against them. Is this something we should change?

There are a number of places in the codebase that use a style attribute on an HTML element rather than styling through CSS.

• app/views/categories/_form.html.erb
• app/views/layouts/_matomo.html.erb
• app/views/posts/_flags_modal.html.erb
• app/views/posts/_flags_modal.html.erb
• app/views/users/mod_privileges.html.erb
• public/assets/community/judaism.js

Some of these are conditional styles, but they could just as easily be conditional classes.

None of these appear to be causing any problems, so this is just to keep track of them to be addressed at some point if it is decided to change the style guide.