Skip to content

Update Blazor binding content#20267

Merged
guardrex merged 2 commits intomasterfrom
guardrex-patch-1
Oct 22, 2020
Merged

Update Blazor binding content#20267
guardrex merged 2 commits intomasterfrom
guardrex-patch-1

Conversation

@guardrex
Copy link
Copy Markdown
Collaborator

@guardrex guardrex commented Oct 22, 2020

Fixes #20263

Internal Review Topic

Thanks @mdekok! 🚀

Let's work this further ...

  • Do we want Task-return or async Task-await binding examples?

  • I don't like the first sentence on Line 130. Can we do something to make it plainer/simpler?

    A common scenario is chaining a data-bound parameter to a page element in the component's output. This scenario is called a chained bind because multiple levels of binding occur simultaneously.

    How about? ...

    A common scenario is chaining a data-bound parameter of a component to a property of field of another component. This scenario is called a chained bind because multiple levels of binding occur simultaneously.

  • Do I (finally 😓) have the headings right with the right examples under the right headings?

Copy link
Copy Markdown
Contributor

@captainsafia captainsafia left a comment

Choose a reason for hiding this comment

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

Left some thoughts to your questions inline.

Comment thread aspnetcore/blazor/components/data-binding.md Outdated
Comment thread aspnetcore/blazor/components/data-binding.md Outdated
Comment thread aspnetcore/blazor/components/data-binding.md Outdated
Comment thread aspnetcore/blazor/components/data-binding.md Outdated
Co-authored-by: Safia Abdalla <safia@microsoft.com>
@guardrex
Copy link
Copy Markdown
Collaborator Author

Excellent ... thank you @captainsafia for helping me sort this out.

WRT async Task-await (instead of Task-return), I think that's something that I'll need to update across the docs in a few other spots, so I'm going to open a separate issue and knock that out fairly fast ... probably tomorrow morning.

Thanks again @mdekok! 🌴

@guardrex guardrex merged commit 0305c3a into master Oct 22, 2020
@guardrex guardrex deleted the guardrex-patch-1 branch October 22, 2020 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Parent-to-child binding example: EventCallback confusing/not needed.

2 participants