Confirmed this builds and runs. It appears to behave correctly in firefox with 123.0 (64-bit). It didn't appear to break anything.
Why we should do this:
This is more semantic.
I think this is better for accessibility, because it will get picked up as a clickable item by screen readers, and it can be accessed by cycling through clickable elements with the Tab key.
This is a more conventional user experience: some users expect a website to allow them to "middle click" clickable elements (the changes in this PR allows "middle click" functionality).
I don't know if there's any reason why we should not do this.
Testing done:
Confirmed this builds and runs. It appears to behave correctly in firefox with 123.0 (64-bit). It didn't appear to break anything.
Why we should do this: