Skip to content

Commit 5fb94be

Browse files
csharpfritzCopilot
andcommitted
docs(skills): Clarify DetailsView+BoundField full support
- Add complete DetailsView+BoundField example in bwfc-migration skill - Add note confirming IColumnCollection implementation works - Emphasize: do NOT replace with HTML tables - Both migration skills now have clear guidance Fixes misconception from Run 11 report that claimed partial support. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 964af2d commit 5fb94be

2 files changed

Lines changed: 28 additions & 1 deletion

File tree

  • .github/skills/webforms-migration
  • migration-toolkit/skills/bwfc-migration

.github/skills/webforms-migration/SKILL.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -443,6 +443,8 @@ Data controls require additional changes for data binding:
443443
- BoundField, TemplateField, and other column types work inside `<Fields>` just like Web Forms
444444
- Child components (BoundField, TemplateField) **inherit the type parameter** via Blazor's `[CascadingTypeParameter]` — no `ItemType` needed on each child
445445

446+
> **✅ FULL SUPPORT:** DetailsView **fully supports** BoundField columns via the `<Fields>` wrapper. BWFC's DetailsView implements `IColumnCollection<ItemType>` which allows BoundField and other column types to register correctly. Do NOT replace DetailsView with HTML tables — use the BWFC component.
447+
446448
#### Repeater
447449

448450
```razor

migration-toolkit/skills/bwfc-migration/SKILL.md

Lines changed: 26 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -940,7 +940,32 @@ These errors frequently occur during migration. Know how to fix them quickly.
940940
</GridView>
941941
```
942942

943-
**Also applies to:** DetailsView needs `<Fields>` wrapper.
943+
**Also applies to:** DetailsView needs `<Fields>` wrapper:
944+
945+
```razor
946+
@* Web Forms — DetailsView with BoundField columns *@
947+
<asp:DetailsView ID="studentData" runat="server"
948+
ItemType="ContosoUniversity.Models.Student"
949+
SelectMethod="GetStudent"
950+
AutoGenerateRows="False">
951+
<Fields>
952+
<asp:BoundField DataField="FirstName" HeaderText="First Name" />
953+
<asp:BoundField DataField="LastName" HeaderText="Last Name" />
954+
<asp:BoundField DataField="EnrollmentDate" HeaderText="Enrolled" DataFormatString="{0:d}" />
955+
</Fields>
956+
</asp:DetailsView>
957+
958+
@* Blazor — DetailsView FULLY SUPPORTS BoundField via <Fields> wrapper *@
959+
<DetailsView Items="@(new[] { student })" ItemType="Student" AutoGenerateRows="false">
960+
<Fields>
961+
<BoundField ItemType="Student" DataField="FirstName" HeaderText="First Name" />
962+
<BoundField ItemType="Student" DataField="LastName" HeaderText="Last Name" />
963+
<BoundField ItemType="Student" DataField="EnrollmentDate" HeaderText="Enrolled" DataFormatString="{0:d}" />
964+
</Fields>
965+
</DetailsView>
966+
```
967+
968+
> **⚠️ NOTE:** DetailsView **fully supports** BoundField. Do NOT replace with HTML tables. The library implements `IColumnCollection<ItemType>` which allows BoundField and other column types to register correctly inside `<Fields>`.
944969
945970
### RZ10001: Type cannot be inferred
946971

0 commit comments

Comments
 (0)