search

leocb / MaterialSkin

Theming .NET WinForms, C# or VB.Net, to Google's Material Design Principles.
MIT License
437 stars 132 forks source link

MaterialRadioButton TabStop Issue #333

Open byungmeo opened 2 years ago

byungmeo commented 2 years ago

image 2022-02-09 16-49-44

if change TabPanelSelectedIndex, MaterialRadioButton Clicked that have first TabIndex among all components in TabPanel. i checked, it actually raise MaterialRadioButton.Click Event

Temporarily, i try first TabIndex was given to other Component and resolve it..

valimaties commented 2 years ago

Hi @byungmeo I was able to reproduce this issue in MaterialExample form, but in another new form does not do the same. So issue has something to do with this example form or in addition to other controls in form. I've noticed another issue, about checkbox... and it can be visible in this gif attached.

RadioButtonIssue

byungmeo commented 2 years ago

@valimaties i think this issue is TabPage & MaterialRadioButton to be associated. because, i just before tried same test like you (RadioButton on MaterialCard? is right?) and does not have problem.

valimaties commented 2 years ago

Ok, I will investigate...

valimaties commented 2 years ago

It was partially fixed in a PR. The only problem is, if you have 2 or more RadioButtons in TabPage and when you activate this TabPage if you have a disabled checked RadioButton, the one which has TabIndex = 0 will get the checked state. I don't know yet why, but I try to figure it out why this happens and how to prevent changing automatically checked state from a disabled RadioButton to an enabled RadioButton.