Wrap FA icons in <button>

[TheTeaCat]

There's a lot of places where buttons are actually just FA icons with tabindexes set on them. These should really be wrapped in a button.

#34 (comment)

[freddyheppell]

Also, the colour themes picker should just use normal radio buttons. I'd suggest either styling the radio buttons to be black and white, or using hidden radio buttons which are still visible to screenreaders and then fa icons to indicate the status of the hidden radio button.