-
Notifications
You must be signed in to change notification settings - Fork 25.1k
Add Blazor with .NET on Web Workers article #36378
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
base: main
Are you sure you want to change the base?
Conversation
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.
Pull Request Overview
This PR adds a new Blazor-specific article about using .NET on Web Workers and updates the existing React-focused article to clarify its scope. The changes enable better documentation organization by separating Blazor and React implementations while maintaining cross-references between them.
Key changes:
- New article explaining Web Workers integration in Blazor WebAssembly apps with QR code generation example
- Updated React article description and sample references to clarify React-specific focus
- Changed CLI sample creation from
-n|--nameto-o|--outputparameter for consistency
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| aspnetcore/blazor/blazor-with-dotnet-on-web-workers.md | New comprehensive article demonstrating Web Workers in Blazor WebAssembly with code examples for QR generation |
| aspnetcore/client-side/dotnet-on-webworkers.md | Updated to clarify React-specific focus, fixed sample app name, changed CLI parameter from -n to -o |
| aspnetcore/toc.yml | Added TOC entry for new Blazor Web Workers article under WebAssembly section |
Co-authored-by: Copilot <[email protected]>
ilonatommy
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.
Looks good
I don't have any preference, I didn't even notice that. You're right that |
Fixes #36370
Ilona ...
-o|--outputfor sample creation.-n|--namewas in the raw content and used for the React article. I'm moving both to-o. Let me know if you think that's a poor choice. I know-ocreates the folder and names the app. I thought-nonly names the app. Truthfully tho, I only really ever used-o.Internal previews