DataGridCheckColumn does not use Theme.ColorOptions.Primary

[oaldrian]

Describe the bug
According to @stsrki DataGridCheckColumn should use Theme.ColorOptions.Primary as base of its color.
This does not work.

To Reproduce
see reproduction in https://github.com/oaldrian/BlazoriseReproductions/tree/datagrid-checkcolumn-color
Index.razor and App.razor

Expected behavior
DataGridCheckColumn should use Theme.ColorOptions.Primary as base of its color.

Screenshots

Additional context
Tested on 1.3

[oaldrian]

seems to also happen for the normal check component, when there is no specific color set.

[stsrki]

I might inform you wrong in the Discord channel.

Can you try this?

[oaldrian]

could be closed as InputOption does solve the problem.

But I would suggest to do an overhaul of the theming section of the docs, to actually include all needed options.

[stsrki]

@steal4life We need to create new API page for the theming options where all the fields will be mentioned.

[tesar-tech]

Just to clarify—are we aiming to bring all the classes from the Themes folder to api docs (as we have it for components)

This will require a few updates, but it doesn't seem too complicated.

Also, is there any other part of the Blazorise source code that we should consider including in the api docs? If so, I can work on creating a more general solution for it, saving time for the future.

[stsrki]

Just to clarify—are we aiming to bring all the classes from the Themes folder to api docs (as we have it for components)

This will require a few updates, but it doesn't seem too complicated.

Also, is there any other part of the Blazorise source code that we should consider including in the api docs? If so, I can work on creating a more general solution for it, saving time for the future.

It wouldn't hurt to have them all listed.

I can think of Chart options. Did we have some problem with those?

[tesar-tech]

Did we have some problem with those?

I think we just didn't complete those.

I will look at it.