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 the page doesn't load anything.