-
Notifications
You must be signed in to change notification settings - Fork 27k
docs: Add docs for EVENT_MANAGER_PLUGINS #64453
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
Conversation
| @HostListener('click.debounce.500') | ||
| handleDebouncedClick(): void { | ||
| console.log('Debounced click!'); | ||
| } |
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.
Can we drop that, HostListeners aren't recommended anymore and only kept for backward compatibility
| template: ` | ||
| <input | ||
| type="text" | ||
| (input.debounce.500)="onSearch($event)" |
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.
| (input.debounce.500)="onSearch($event)" | |
| (input.debounce.500)="onSearch($event.target.value)" |
0feeba0 to
3de67fd
Compare
3de67fd to
a7f3754
Compare
|
Updated |
JeanMeche
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.
Thanks for the addition, it's quite an old feature but not well known !
|
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. |
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: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Other information