Skip to content

[Blazor] Rename LinkPreload to ResourcePreloader #62673

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

Merged
merged 2 commits into from
Jul 12, 2025

Conversation

maraf
Copy link
Member

@maraf maraf commented Jul 11, 2025

Rename the component rendering preloading links based on API review.

- <LinkPreload  />
+ <ResourcePreloader />

Fixes #62374

cc @guardrex

@maraf maraf added this to the 10.0-preview7 milestone Jul 11, 2025
@maraf maraf self-assigned this Jul 11, 2025
@Copilot Copilot AI review requested due to automatic review settings July 11, 2025 11:08
@maraf maraf requested a review from a team as a code owner July 11, 2025 11:08
@maraf maraf added the area-blazor Includes: Blazor, Razor Components label Jul 11, 2025
Copy link
Contributor

@Copilot 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

This PR renames the existing LinkPreload component to ResourcePreloader across templates, tests, public API surfaces, and implementation.

  • Updates Blazor Web template and test wrapper markup to use <ResourcePreloader /> instead of <LinkPreload />
  • Adjusts PublicAPI.Unshipped.txt entries to reflect the new ResourcePreloader type
  • Renames the LinkPreload class to ResourcePreloader in its source file

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/ProjectTemplates/Web.ProjectTemplates/content/BlazorWeb-CSharp/BlazorWebCSharp.1/Components/App.razor Replaced <LinkPreload /> with <ResourcePreloader /> in template
src/Components/Endpoints/test/TestComponents/WebAssemblyPreloadWrapper.razor Updated test component to invoke <ResourcePreloader />
src/Components/Endpoints/src/PublicAPI.Unshipped.txt Swapped public API entries from LinkPreload to ResourcePreloader
src/Components/Endpoints/src/Assets/ResourcePreloader.cs Renamed LinkPreload class to ResourcePreloader
Comments suppressed due to low confidence (1)

src/Components/Endpoints/src/Assets/ResourcePreloader.cs:10

  • [nitpick] The XML documentation still refers to "link elements" for preloading; consider updating the summary to align with ResourcePreloader naming, e.g., "Represents a component that renders preload link elements for resources."
/// Represents link elements for preloading assets.

@maraf maraf merged commit f905beb into dotnet:main Jul 12, 2025
29 checks passed
@maraf maraf deleted the BlazorPreloadRename branch July 12, 2025 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Api proposal for Use <LinkPreload /> component to preload assets (#62225)
3 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy