Prevent unnecessary VE page load

Fix a bug where the logLinkClick helper, which sets up the logging of
the SignupExpPageLinkClick event, would cause the "Edit" links to reload
the page before activating the Visual Editor.

Additionally, ensure that the SignupExpPageLinkClick event is logged if
the user has already seen the pre-edit CTA.

Bug: 66236
Change-Id: I6c41c1a5915d6d690c4c6efec56c7f8a438b39cc