I think it's best practice to have a fragment identifier (`#`) instead of an empty href attribute when you are using onClick for the link behaviour.