MudSlider: Use ParameterState and generic math

[ScarletKuro]

Description

Uses ParameterState and generic math so we do not need pesky converters

How Has This Been Tested?

Current unit tests & visual

Type 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)
• Documentation (fix or improvement to the website or code docs)

Checklist

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

[codecov]

Codecov Report

Attention: Patch coverage is 77.77778% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 90.08%. Comparing base (28bc599) to head (2c96fe6).
Report is 80 commits behind head on dev.

Files	Patch %	Lines
src/MudBlazor/Components/Slider/MudSlider.razor.cs	82.35%	1 Missing and 2 partials ⚠️
src/MudBlazor/Components/Slider/MudSlider.razor	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #8745      +/-   ##
==========================================
+ Coverage   89.82%   90.08%   +0.25%     
==========================================
  Files         412      419       +7     
  Lines       11878    12174     +296     
  Branches     2364     2396      +32     
==========================================
+ Hits        10670    10967     +297     
+ Misses        681      667      -14     
- Partials      527      540      +13     

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

[ScarletKuro]

MudBlazor/src/MudBlazor.Docs/Services/Menu/MenuService.cs

Line 88 in cd1cb6e

.AddItem("Switch", typeof(MudSwitch<T>))

Because of this line and how docs currently work I had no other ideas but implement INumber to the T class. Obviously this is not the best solution and this methods should not use T : class, because what if someone needs a struct or anything like that. I couldn't even use NotImplementedException because something is invoking the methods.

@henon let me know if there is better ways.

[danielchalmers]

What's the original purpose of the T class?

[ScarletKuro]

What's the original purpose of the T class?

It's used for docs and in some unit tests to construct the component.

[danielchalmers]

lgtm from my limited knowledge on this

[henon]

About class T:
The alternative would be to have a registry of types which can be used with each generic component such as MudSwitch => bool etc and then we can construct the components for the automated tests and for the API docs which use reflection to discover the properties.

[henon]

@henon let me know if there is better ways.

It suffices for now. We'll solve the problem of class T later for good.

[henon]

Thanks, the code looks much better now.

Added to v7.0.0 Migration Guide #8447

[ScarletKuro]

I just thought, would it make sense to add "CultureInfo" parameter in case someone want's the ValueLabel to be in different format, for example have a coma instead of point, show currency symbol if its money related slider etc?

[henon]

Yes, that makes sense.

[ScarletKuro]

#8760