-
Notifications
You must be signed in to change notification settings - Fork 27k
fix(elements): return value on signal input getter #62113
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(elements): return value on signal input getter #62113
Conversation
2eecaf1 to
d216d18
Compare
c63c5ac to
06676ee
Compare
atscott
left a comment
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.
@alxhub I think we would need to consider this a breaking change?
06676ee to
39957d5
Compare
39957d5 to
c83214b
Compare
|
Converting to draft, as we'll look at it again when we enter the v21 breaking change window. |
BREAKING CHANGE: Fix signal input getter behavior in custom elements. Before this change, signal inputs in custom elements required function calls to access their values (`elementRef.newInput()`), while decorator inputs were accessed directly (`elementRef.oldInput`). This inconsistency caused confusion and typing difficulties. The getter behavior has been standardized so signal inputs can now be accessed directly, matching the behavior of decorator inputs: Before: - Decorator Input: `elementRef.oldInput` - Signal Input: `elementRef.newInput()` After: - Decorator Input: `elementRef.oldInput` - Signal Input: `elementRef.newInput` closes angular#62097
c83214b to
1972f9c
Compare
|
This PR was merged into the repository. The changes were merged into the following branches:
|
BREAKING CHANGE: Fix signal input getter behavior in custom elements. Before this change, signal inputs in custom elements required function calls to access their values (`elementRef.newInput()`), while decorator inputs were accessed directly (`elementRef.oldInput`). This inconsistency caused confusion and typing difficulties. The getter behavior has been standardized so signal inputs can now be accessed directly, matching the behavior of decorator inputs: Before: - Decorator Input: `elementRef.oldInput` - Signal Input: `elementRef.newInput()` After: - Decorator Input: `elementRef.oldInput` - Signal Input: `elementRef.newInput` closes angular#62097 PR Close angular#62113
|
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. |
closes #62097
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: #62097
What is the new behavior?
Does this PR introduce a breaking change?
Other information