fix(RouterLink): ignore optional parameters when checking for active …

[vicb]

…routes

• Please check if the PR fulfills these requirements
• The commit message follows our guidelines: https://github.com/angular/angular/blob/master/CONTRIBUTING.md#commit-message-format
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)
• What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

A bug fix

• What is the current behavior? (You can also link to an open issue here)

When the URL has query parameters, the RouterLink directive would not mark the link as active (adding a CSS class)

• What is the new behavior (if this is a feature change)?

The class is added regardless there are query parameters

• Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

No

fixes #6459

[btford]

LGTM

[mary-poppins]

Merging PR #7834 on behalf of @alxhub to branch presubmit-alxhub-pr-7834.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}