activescott / lessmsi

A tool to view and extract the contents of an Windows Installer (.msi) file.
https://lessmsi.activescott.com
MIT License
1.29k stars 150 forks source link

Some small usability fixes #155

Closed learn-more closed 3 years ago

learn-more commented 3 years ago

Split up in individual commits per feature, if you disagree with any of those please let me know and I can drop it from the PR.

activescott commented 3 years ago

Thanks again @learn-more! I don't see anything wrong here, but I'll have to take a closer look at this one and play with it. Again, I should be able to either merge it or give you some feedback this week or over the upcoming weekend.

learn-more commented 3 years ago

@activescott I have addressed your comments.

For the sorting: I did agree that this was needlessly verbose in the place where it was, so I moved the code to an extension function. The sorting is now down in the way you suggested.

activescott commented 3 years ago

Dope. Love the DataGridviewExtentionMethods approach. Well done!

activescott commented 3 years ago

:tada: This PR is included in version 1.8.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: