zap.stream icon indicating copy to clipboard operation
zap.stream copied to clipboard

Substitute html Link for onClick handler on Logo

Open kuiperanon opened this issue 10 months ago • 0 comments

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:

  • 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.

kuiperanon avatar Mar 26 '24 16:03 kuiperanon