Skip to content

Conversation

@mjrousos
Copy link
Member

.NET Core 3 WinForms sample demonstrating a variety of DataGridView scenarios. Ported from https://code.msdn.microsoft.com/windowsdesktop/CSWinFormDataGridView-29783221


/// <summary>
/// Returns an image deserialized from a string resource to work around https://github.com/dotnet/corefx/issues/26745
/// </summary>
Copy link
Member Author

Choose a reason for hiding this comment

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

Now that there's a fix for dotnet/corefx#26745 in the works, I can probably remove this.

Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

This looks good, @mjrousos
A couple of additional issues:

  • Microsoft-related URLs should begin with https:// rather than http://
  • README.md should be updated to include this sample.

@mjrousos
Copy link
Member Author

@rpetrusha, I've updated the PR based on your comments. Any other suggestions here or is this one ready to merge?

@rpetrusha
Copy link

Thanks for updating your PR, @mjrousos. This is ready to merge.

@rpetrusha rpetrusha merged commit a6d77fa into dotnet:master Nov 19, 2018
@mjrousos mjrousos deleted the mikerou/DataGridView branch November 21, 2018 21:20
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