Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f6b563b) 88.07% compared to head (371d22e) 88.08%.

@@           Coverage Diff           @@
##              dev    #8010   +/-   ##
=======================================
  Coverage   88.07%   88.08%           
=======================================
  Files         393      393           
  Lines       11742    11748    +6     
  Branches     2375     2376    +1     
=======================================
+ Hits        10342    10348    +6     
  Misses        874      874           
  Partials      526      526           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

If this PR is accepted, I will do a similar PR to Items and Groupable that don't work with ServerData.
Finally, a PR to add a documentation about ServerData with some best practice.

@deinok, this PR revert your PR #4664. If you have a better suggestion to fix the problem and keep the feature, can you describe it in a comment?

@henon, can you review this PR?

@tjscience FYI

Normally you would query the DataSource remotely to apply the filter wouldn't you?

Yes, the quickfilter would only give you a filtered view of the current page which is not what you would expect.

@mikes-gh :

Normally you would query the DataSource remotely to apply the filter wouldn't you?

Yes, with ServerData all the logic must be handled externally by backend or database.

ServerData allows to load only the data displayed on the current page. It's very convenient when you work with huge datasets, because you avoid to load all elements.

QuickFilter is a delegate what work like Where in Linq to Object. So to work, it need to load all elements in memory.
The original bug, it's QuickFilter with ServerData filter only the current page... because only elements of the current page are loaded.

A solution is to load all elements and apply QuickFilter, but this loses the point of ServerData to load partially the data.
Why I suggest to disable QuickFilter when ServerData is the provider.

If you want use QuickFilter, then load all elements and set the parameter DataGrid.Items.

Thanks @vernou !

Hey @vernou did you make progress on the ServerData example for the DataGrid since there currently is none.
Would be great to get some best practices that I can start working from. ✌️

If this PR is accepted, I will do a similar PR to Items and Groupable that don't work with ServerData. Finally, a PR to add a documentation about ServerData with some best practice.

Also the additional exceptions would be great. Forgot to remove mine as I am switching to ServerData for some of my DataGrids.

From the PR #8026, when ServerData and Items are mixed, it throw a runtime exception to advice the developer.

After I go on other thing and forget the parameter Groupable and to improve the documentation.
Nice callback.

@dennisrahmen
Groupable work fine with ServerData.

You can found example to use ServerData on the doc of MudTable. It's very similar.

@vernou I know the example of the MudTable using that for reference in my projects.

I just had a colleague at work completely replace all DataGrids in a Project since he thought only MudTable had a ServerData example.

I was on vacation that week. :(