Skip to content

Conversation

@guardrex
Copy link
Collaborator

@guardrex guardrex commented Oct 23, 2025

Fixes #36052

Distilled from Javier's remarks at dotnet/aspnetcore#57962 (comment).

Tom or Wade: Ready here! This is just mirroring some really nice remarks that Javier made in a PU discussion ☝️ that devs will likely find helpful in understanding why they shouldn't write directly to component parameters, favoring @bind:get/set directive attributes instead. This adds a little more juice to the existing section and example on binding across components. I only need one review to get this in.

For reference, here's one of the examples (because you can't see them here with the cross-repo links) ...

https://github.com/dotnet/blazor-samples/blob/main/10.0/BlazorSample_BlazorWebApp/Components/NestedChild.razor


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/components/data-binding.md aspnetcore/blazor/components/data-binding

@guardrex guardrex self-assigned this Oct 23, 2025
@guardrex guardrex requested a review from Copilot October 23, 2025 14:42

This comment was marked as outdated.

@guardrex guardrex requested a review from Copilot October 30, 2025 13:18
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Copy link
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

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

Looks good! Approved.

@guardrex guardrex merged commit 2658ee4 into main Oct 30, 2025
10 checks passed
@guardrex guardrex deleted the guardrex/blazor-two-way-binding branch October 30, 2025 18:28
@guardrex guardrex removed the request for review from tdykstra October 30, 2025 18:29
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.

Potential further improvements to multiple level binding scenario coverage

3 participants