Skip to content
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

fix(core): inherit host directives #52992

Closed

Conversation

crisbeto
Copy link
Member

Adds support for inheriting host directives from the parent class. This is consistent with how we inherit other features like host bindings.

Fixes #51203.

@crisbeto crisbeto added action: review The PR is still awaiting reviews from at least one requested reviewer area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release action: global presubmit The PR is in need of a google3 global presubmit labels Nov 17, 2023
@ngbot ngbot bot modified the milestone: Backlog Nov 17, 2023
@crisbeto crisbeto removed the action: global presubmit The PR is in need of a google3 global presubmit label Nov 17, 2023
@crisbeto
Copy link
Member Author

Passing TGP

@crisbeto crisbeto marked this pull request as ready for review November 17, 2023 12:55
@crisbeto
Copy link
Member Author

I realized that this will need a bit more work on the compiler side to properly support template type checking. Putting it back to draft.

@crisbeto crisbeto marked this pull request as draft November 17, 2023 14:38
@crisbeto crisbeto force-pushed the 51203/host-directives-inherit branch from a20048a to bc48a49 Compare November 17, 2023 20:23
@crisbeto
Copy link
Member Author

Okay, now everything should be covered.

@crisbeto crisbeto marked this pull request as ready for review November 17, 2023 20:24
Adds support for inheriting host directives from the parent class. This is consistent with how we inherit other features like host bindings.

Fixes angular#51203.
@crisbeto crisbeto force-pushed the 51203/host-directives-inherit branch from bc48a49 to c4f4eb5 Compare November 17, 2023 20:25
Copy link
Contributor

@jessicajaniuk jessicajaniuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jessicajaniuk jessicajaniuk removed the action: review The PR is still awaiting reviews from at least one requested reviewer label Nov 20, 2023
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Nov 20, 2023
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit c7c7ea9.

AndrewKushnir pushed a commit that referenced this pull request Nov 20, 2023
Adds support for inheriting host directives from the parent class. This is consistent with how we inherit other features like host bindings.

Fixes #51203.

PR Close #52992
tbondwilkinson pushed a commit to tbondwilkinson/angular that referenced this pull request Dec 6, 2023
Adds support for inheriting host directives from the parent class. This is consistent with how we inherit other features like host bindings.

Fixes angular#51203.

PR Close angular#52992
@angular-automatic-lock-bot
Copy link

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.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 21, 2023
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
Adds support for inheriting host directives from the parent class. This is consistent with how we inherit other features like host bindings.

Fixes angular#51203.

PR Close angular#52992
rlmestre pushed a commit to rlmestre/angular that referenced this pull request Jan 26, 2024
Adds support for inheriting host directives from the parent class. This is consistent with how we inherit other features like host bindings.

Fixes angular#51203.

PR Close angular#52992
amilamen pushed a commit to amilamen/angular that referenced this pull request Jan 26, 2024
Adds support for inheriting host directives from the parent class. This is consistent with how we inherit other features like host bindings.

Fixes angular#51203.

PR Close angular#52992
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HostDirectives doesnt provide the hosted directives when inherited
3 participants