This came up in D3364 when we manually added `cursor:pointer` to `a.sendButton`.
Followed @palys-swm suggestion (https://phabricator.ashoat.com/D3364#90810) and applied this style "globally."
Note: I am **not** planning on going through the codebase at this moment and cutting every extraneous `cursor:pointer` applied to an `<a>` or `<button>`. I think doing so would be fine, but I don't want to check every single instance right now to ensure that there are no regressions. I think it's fine to leave them in even if they may be redundant.