Do a pass over the app and update the role attribute

[camargo]

We are using role="none" in some places which is bad for accessibility. This ticket is to do a pass over the app and use a proper role. We should also convert elements to buttons if they are actual buttons.