Describe the issue:

Right now, when table headers do not have actions, they are rendered with the button tag, but given role=presentation. The issue is that this is not a valid role for a button element (https://w3c.github.io/html-aria/#el-button).

Presentation/none roles are especially prone to bugs when applied to natively focusable and interactive elements, since browsers and screen readers sometimes attempt to correct that type of author error.

Expected behavior:

We should switch to a div or span (maybe span to keep the default inline display styling?) when the header does not need to perform an action

cc / @ling1726