GridView improvements: Change rendering ownership; add HyperLinkField

FritzAndFriends / BlazorWebFormsComponents

A collection of Blazor components that emulate the ASP.NET Web Forms controls of the same name

MIT License

428 stars 72 forks source link

GridView improvements: Change rendering ownership; add HyperLinkField #215

Closed Eilon closed 4 years ago

Eilon commented 4 years ago

This change is split into 2 commits to make it easier to review.

Eilon commented 4 years ago

For the whitespace: For some reason VS is having a fit on doing consistent whitespace in this solution. I feel like several files are already inconsistent. Would it make sense to send a separate PR with a full solution-wide cleanup of all whitespace?

csharpfritz commented 4 years ago

Looks really good @Eilon -- can we add a unit test for the HyperLinkField?

Eilon commented 4 years ago

What is unit test?

Eilon commented 4 years ago

Just kidding I'll add something

Eilon commented 4 years ago

Updated based on feedback.

csharpfritz commented 4 years ago

LGTM