New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(router): merge RouterLinkWithHref
into RouterLink
#47630
Conversation
2d51b7e
to
0716068
Compare
This commit updates the `RouterLink` to extend the selector to also include `<a>` and `<area>` elements, which were previously matched by the `RouterLinkWithHref` directive. The code of the directives was merged together (since there was a lot of duplication) and this commit finalizes the merge. The `RouterLinkWithHref` becomes an alias of the `RouterLink` directive. Now there is no need to import and use the `RouterLinkWithHref` class, the `RouterLink` directive will match any element that has the `routerLink` attribute. DEPRECATED: The `RouterLinkWithHref` directive is deprecated, use the `RouterLink` directive instead. The `RouterLink` contains the code from the `RouterLinkWithHref` to handle elements with `href` attributes.
0716068
to
fb06e58
Compare
@@ -33,7 +33,7 @@ | |||
"cli-hello-world-lazy": { | |||
"uncompressed": { | |||
"runtime": 2835, | |||
"main": 226404, | |||
"main": 225113, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥳
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed-for: public-api, size-tracking
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed-for: public-api, size-tracking
Note to self: this is blocked by the internal cleanup. |
Merge-assistance: internal cleanup is completed, this PR is ready for merge. Please merge and sync on its own (to simplify a rollback if needed). |
This PR was merged into the repository by commit f73ef21. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This commit updates the
RouterLink
to extend the selector to also include<a>
and<area>
elements, which were previously matched by theRouterLinkWithHref
directive. The code of the directives was merged together (since there was a lot of duplication) and this commit finalizes the merge. TheRouterLinkWithHref
becomes an alias of theRouterLink
directive.Now there is no need to import and use the
RouterLinkWithHref
class, theRouterLink
directive will match any element that has therouterLink
attribute.PR Type
What kind of change does this PR introduce?
Does this PR introduce a breaking change?