Proposal bind shorthand #37
Conversation
# Conflicts: # force-app/lwc/signals/core.js # src/lwc/signals/core.ts
| } | ||
| } | ||
|
|
||
| function $binded(component: Record<string, object>, propertyName: string) { |
There was a problem hiding this comment.
Rename to just bind (or bound)
There was a problem hiding this comment.
go with bind it's pretty standard
| import { counter, counterPlusTwo } from "c/demoSignals"; | ||
|
|
||
| export default class CountTracker extends LightningElement { | ||
| bindTest = $binded(this, "bindTest").to(counter); |
There was a problem hiding this comment.
I understand why you are doing this but I don't love it. Not sure what would be a better way to do the binding
There was a problem hiding this comment.
Yeah, that's why I'm keeping it as a proposal stage until I get some feedback, not sure if you saw the issue I put up for discussion: #38
I don't want to merge if its going to be detrimental compare to what exists today, but I haven't found a better way of doing it with how Javascript works today. The only way I can think this could be done is by using decorators, but JS doesn't support those yet.
| this.propertyName = propertyName; | ||
| } | ||
| to(signal) { | ||
| $effect(() => { |
There was a problem hiding this comment.
is this ok performance wise?
No description provided.