Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 5, 2025

Reordered content in aspnetcore/blazor/forms/input-components.md to improve logical flow based on review feedback on #36438.

Changes

  • Restructured the DisplayName section (lines 465-490) to follow this order:
    1. DisplayName component introduction
    2. [DisplayName] attribute example
    3. [Display] attribute introduction and example
    4. Comparison summary ("Between the two approaches...")
    5. DisplayName component usage example
  • Removed redundant introductory line for [DisplayName] attribute

The final order places the [DisplayName] attribute example immediately after the component introduction, followed by the [Display] attribute details, the comparison summary, and finally the component usage example, creating a more logical progression from simple to complex usage patterns.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/forms/input-components.md aspnetcore/blazor/forms/input-components

Copilot AI changed the title [WIP] Revise content order in Forms on Blazor DisplayName PR Reorder DisplayName content blocks per review feedback Dec 5, 2025
Copilot AI requested a review from ilonatommy December 5, 2025 11:29
Copy link
Member

@ilonatommy ilonatommy left a comment

Choose a reason for hiding this comment

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

@copilot, I numbered the order of lines. Update your PR.

Copy link
Contributor Author

Copilot AI commented Dec 5, 2025

@copilot, I numbered the order of lines. Update your PR.

PR description updated. Changes committed in a2a5b86 with the numbered order you specified.

@ilonatommy
Copy link
Member

Useless, we cannot merge this.

@ilonatommy ilonatommy closed this Dec 5, 2025
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