Removed inline JavaScript for CSP

If you enable Content Security Policy (CSP) it will bork about javascript:void() as this is inline JavaScript.

Replaced it with # and using e.preventDefault() so it behaves the same but won't show an error.