DigitalAdeel / MaterialSkin2DotNet

62 stars 18 forks source link

Scrolling within an empty datatable leads to System.ArgumentOutOfRangeException #11

Closed Dr33v3s closed 1 year ago

Dr33v3s commented 1 year ago

Hey man, great work. I've only been using it for a short time but everything looks great and works as expected.

But when i scroll the mouse wheel in a datatable with no values, I get a System.ArgumentOutOfRangeException.

DigitalAdeel commented 1 year ago

@Dr33v3s Hi and thanks for reporting :) I'll add a null/count checker for it.

DigitalAdeel commented 1 year ago

@Dr33v3s Glad that you liked something that I developed cuz the original author didn't 😅 Nowadays, I'm busy doin freelancing projects, that's why I didn't do any bug fixing or added new feature.

DigitalAdeel commented 1 year ago

@Dr33v3s Lemme know if you got any feature request or something cuz I might do it in my free time ;)

Dr33v3s commented 1 year ago

@Dr33v3s Hi and thanks for reporting :) I'll add a null/count checker for it.

Thanks for that, I appreciate the effort and care for your work.

@Dr33v3s Lemme know if you got any feature request or something cuz I might do it in my free time ;)

I'm perfectly happy, but it's great to see that you help others with your efforts. And all of that in your free time. Again thank you.

DigitalAdeel commented 1 year ago

@Dr33v3s De Nada Hermano :)

DigitalAdeel commented 1 year ago

@Dr33v3s Debugging :)

image

DigitalAdeel commented 1 year ago

@Dr33v3s Fixed 😉 image

DigitalAdeel commented 1 year ago

@Dr33v3s The issue has been fixed in commit #a6ad72a . The changes were made to the file MaterialDataTable.cs

Dr33v3s commented 1 year ago

Thank you for fixing the problem so quickly. And of course thank you very much for the credit.

You‘re a great guy.

DigitalAdeel commented 1 year ago

Thank you for fixing the problem so quickly. And of course thank you very much for the credit.

You‘re a great guy.

De Nada Amigo :)

FYI: Working on Pagination for MaterialDataTable😉

image