fix: remove link handler in disconnect

[xuwaters]

Pull Request

📖 Description

Click event handler should be removed properly in disconnect method of DefaultLinkHandler.
Since the event listener is added with useCapture = true in connect method, it's required to set useCapture = true in removeEventListener according to https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/removeEventListener#matching_event_listeners_for_removal

🎫 Issues

When the :config property of fast-router is updated, the tryConnect method will disconnect link handler and connect again. In this case, the previous click event handler is not properly removed, which results in resource leak.

👩‍💻 Reviewer Notes

📑 Test Plan

✅ Checklist

General

• I have included a change request file using $ yarn change
• I have added tests for my changes.
• I have tested my changes.
• I have updated the project documentation to reflect my changes.
• I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

• I have added a new component
• I have modified an existing component
• I have updated the definition file
• I have updated the configuration file

⏭ Next Steps

[scomea]

This looks like the right thing to do, but regardless there should be some explanatory description in the PR.

[xuwaters]

This looks like the right thing to do, but regardless there should be some explanatory description in the PR.

Description updated.