MudBlazor / MudBlazor

Blazor Component Library based on Material design with an emphasis on ease of use. Mainly written in C# with Javascript kept to a bare minimum it empowers .NET developers to easily debug it if needed.
http://mudblazor.com
MIT License
8.04k stars 1.28k forks source link

MudDataGrid needs a SortChanged event that can be subscribed to. #6433

Open tjscience opened 1 year ago

tjscience commented 1 year ago

Feature request type

Enhance component

Component name

MudDataGrid

Is your feature request related to a problem?

No

Describe the solution you'd like

Currently, there is no way to easily tell when the sort has changed in the data grid. There is a SortDefinitions parameter but the type is an Dictionary<string, SortDefinition> which does not allow notifying when a definition has been added, removed or changed. There needs to be an event, SortChanged than can be subscribed to for notifying when the sort of the data grid has changed.

Have you seen this feature anywhere else?

No

Describe alternatives you've considered

The only solution really is implementing your own ObservableDictionary.

Pull Request

Code of Conduct

dev-jlb commented 1 year ago

Seconded!

CristianSala76 commented 3 months ago

Hi, will this event be available in the next future? I see SortChangedEvent in the Documentation API but this event is not public thank you

ScarletKuro commented 3 months ago

Hi, will this event be available in the next future?

Pull Request is welcome.

CristianSala76 commented 3 months ago

Hi, will this event be available in the next future?

Pull Request is welcome.

sorry, what do you mean?

ScarletKuro commented 3 months ago

sorry, what do you mean?

MudBlazor is a community project. When someone needs a feature, users like you can implement it, post the code through a pull request, and then the team will review it. If everything is okay, the code will be added to the MudBlazor codebase.

CristianSala76 commented 3 months ago

MudBlazor is a community project. When someone needs a feature, users like you can implement it, post the code through a pull request, and then the team will review it. If everything is okay, the code will be added to the MudBlazor codebase.

it's ok, but, as the documentation says that there is such an event, I think that this is a bug either in the code or in the docs.

ScarletKuro commented 3 months ago

it's ok, but, as the documentation says that there is such an event, I think that this is a bug either in the code or in the docs.

It's bug in the docs, it's not suppose to show internal api, currently the API docs going through a rewrite, so yeah there are some bugs and missing features in the docs.