Skip to content

Conversation

@hakenr
Copy link
Member

@hakenr hakenr commented Oct 23, 2024

When reading this article, I found it difficult to understand the meaning of:

  • In server-side apps with multiple users concurrently using the same component.

Eventually, I realized it refers to the difference between calling static .NET methods (where any static state is shared across circuits) and instance .NET methods (where the instance is held by the component, which is part of an individual circuit).

I think updating the introductory sentence would help future readers better understand this context.


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/javascript-interoperability/call-dotnet-from-javascript.md aspnetcore/blazor/javascript-interoperability/call-dotnet-from-javascript

@hakenr hakenr requested a review from guardrex as a code owner October 23, 2024 22:47
@hakenr hakenr changed the title [Blazor] Calling .NET from JS - explanation scenarios for DotNetObjectReference and instance methods [Blazor] Calling .NET from JS - explanation of scenarios for DotNetObjectReference and instance methods Oct 23, 2024
@guardrex guardrex self-assigned this Oct 23, 2024
Copy link
Collaborator

@guardrex guardrex left a comment

Choose a reason for hiding this comment

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

👍 I agree with the improvement.

Would you mind fixing a line that I missed earlier? On Line 1179, it states ...

  • The component contains several ListItem1 components, which is a shared component in the app's Shared folder.

Would you simplify that to ...

  • The component contains several ListItem1 components.

I'd prefer not to version the remark for the location difference between <8.0 and >=8.0. That's the simplest way to handle it.

Nevermind! 😄

I got it. We're good!

@guardrex
Copy link
Collaborator

Actually ... I'll do it. I'll shoot that change in from here.

@guardrex guardrex enabled auto-merge (squash) October 23, 2024 23:16
@guardrex guardrex merged commit 433b86e into dotnet:main Oct 23, 2024
2 checks passed
@hakenr hakenr deleted the patch-35 branch October 24, 2024 00:13
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.

2 participants