Skip to content

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.

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