Skip to content
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

Blazor overview updates 8.0 #30353

Merged
merged 9 commits into from
Sep 27, 2023
Merged

Blazor overview updates 8.0 #30353

merged 9 commits into from
Sep 27, 2023

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented Sep 14, 2023

Fixes #30285
Addresses #28161

Notes

  • When 🔪 here, I made sure that the Hosting Models and Components overview received a couple of items that they didn't have to ensure that we don't lose any coverage. The entire Components section of the Blazor overview can be 🔪 because we have all of that coverage back in the Components overview article. Assuming that you like the cut, I dropped the content for all versions, but do you want to keep the components coverage for <8.0 coverage?
  • I merged the prior Blazor Web Apps and Blazor WebAssembly content with an emphasis on the former.
  • For >=8.0, we don't have a BWA image, and I didn't keep the Blazor WASM image for >=8.0 coverage. Do you want any images in >=8.0 coverage?

Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/components/index.md ASP.NET Core Razor components
aspnetcore/blazor/hosting-models.md ASP.NET Core Blazor hosting models
aspnetcore/blazor/index.md ASP.NET Core Blazor
aspnetcore/release-notes/aspnetcore-3.0.md aspnetcore/release-notes/aspnetcore-3.0
aspnetcore/tutorials/choose-web-ui.md Choose an ASP.NET Core web UI

Copy link
Member

@danroth27 danroth27 left a comment

Choose a reason for hiding this comment

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

I have a few suggestions for your consideration.

Also, can we include in this PR a cleanup pass on the two bulleted lists at the beginning of the doc? The two lists seem pretty repetitive. I think the first list should summarize what Blazor is, and the second list should enumerate its advantages. I also want to stop using the phrase "instead of JavaScript" to avoid alienating frontend JavaScript devs that might be interested in Blazor. Instead of saying "instead of JavaScript" we should focus on how Blazor makes building web apps easier and more productive.

@guardrex
Copy link
Collaborator Author

@danroth27 ... Updates made ... ready for another 👀.

Copy link
Member

@danroth27 danroth27 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 to me. Thanks!

@guardrex guardrex merged commit e8ce576 into main Sep 27, 2023
2 checks passed
@guardrex guardrex deleted the guardrex/blazor-overview branch September 27, 2023 15:12
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.

Consolidate and unify the Blazor Web App and Blazor WebAssembly App content on the Blazor overview page
2 participants