Artyom M.

Results 734 comments of Artyom M.

Closing and there is no feedback from the author.

I'm not sure about this change, it's kinda breaking change because it changes the public api `AddFilter` from void to Task. Also, I don't really understand the moment with commenting...

> Not sure why they chose this instead of just using `justify-center`. Do you mind creating a pull request? > it would be a breaking change, although honestly I don't...

@Yomodo can you explain this since you added this to the documentation? @henon maybe you can help as well.

Oh ok, I just see that this sentence was added in your PR https://github.com/MudBlazor/MudBlazor/pull/1810 But now i read that you took the sentence from the material documentation

Hi @daniellor Some tests are failing that are related to the datagrid. Also I don't see any explicitly added tests for this feature, this is a must for such complicated...

It was Garderoben who contributed it. From my experience you can either put a "global" `MudOverlay` somewhere in the `MainLayout`. or have it on components, for example `MudPaper` etc. In...

@tjscience can you review, please? What do you think of this changes?

Hi. As @tjscience said, this PR looks overly complicated for a simple `ItemsProvider` delegate. Also this PR is now against our contribution guide. https://github.com/MudBlazor/MudBlazor/blob/5499685424bf6c585251f29846737251700bcff6/src/MudBlazor/Components/Virtualize/MudVirtualize.razor.cs#L53-L76 We do not add logic to...