MudDataGrid: Add FilterVariant Property

[dunds-com]

Description

MudDataGrid was missing the Variant property, so you could not set it for the filtering.

How Has This Been Tested?

Visually tested using the docs.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• The PR is submitted to the correct branch (dev).
• My code follows the code style of this project.
• I've added relevant tests.

[henon]

Please change the xml comment

[henon]

Can you show before and after screenshots how the new visuals look? Thanks

[dunds-com]

@henon
Old:

New:

[ScarletKuro]

fyi: the tests fail now

[tjscience]

This looks good to me. Thanks, @dunds-com!

[ScarletKuro]

@dunds-com can you fix the tests? It looks related, then we can merge.

[dunds-com]

@ScarletKuro @tjscience @henon

The test doesn't seem to find the add button in the filters anymore.
comp.FindAll(".filters-panel > button")[0].Click();

But it does work when using the method.
await comp.InvokeAsync(() => dataGrid.Instance.AddFilter());

Maybe setting the variant is causing the button to be rendered with a delay?

[henon]

If it is indeed a timing issue as you suspect then you could prove it by waiting for 1 second with await Task.Delay(TimeSpan.FromSeconds(1)). But I am not so sure that's it. Variant shouldn't introduce a delay. Maybe the css selector is not correct any longer?

[dunds-com]

@henon

await Task.Delay(TimeSpan.FromSeconds(1));

// click the Add Filter button in the filters panel to add a filter
comp.FindAll(".filters-panel > button")[0].Click();

Adding the delay does not seem to have an effect.

[henon]

Do a Constole.WriteLine(comp.Markup); to see the html structure at that point in time. That'll tell you why the css selector doesn't work

[ScarletKuro]

Very strange, technically added property shouldn't have affected anything except the .mud-button-filled-primary