Fixes #5040. SelectorBase.TabBehavior setter calls CreateSubViews() without UpdateChecked(), desyncing CheckBox visual state

[BDisp]

Fixes

• Fixes SelectorBase.TabBehavior setter calls CreateSubViews() without UpdateChecked(), desyncing CheckBox visual state #5040

Proposed Changes/Todos

• Call UpdateChecked inside the CreateSubViews method

Pull Request checklist:

• I've named my PR in the form of "Fixes #issue. Terse description."
• My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
• My code follows the Terminal.Gui library design guidelines
• I ran dotnet test before commit
• I have made corresponding changes to the API documentation (using /// style comments)
• My changes generate no new warnings
• I have checked my code and corrected any poor grammar or misspellings
• I conducted basic QA to assure all features are working

[Copilot]

Pull request overview

Fixes the selector visual-state desync reported in #5040 by ensuring checkbox checked state is synchronized after selector subviews are recreated.

Changes:

• Move “sync checked state” behavior to run after CreateSubViews () completes (instead of being manually invoked by several property setters).
• Add logic in OptionSelector.UpdateChecked () to reset Value when it no longer matches any checkbox.
• Add a new unit test covering Value reset when Values changes.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
Terminal.Gui/Views/Selectors/SelectorBase.cs	Removes explicit UpdateChecked () calls from multiple setters and calls UpdateChecked () at the end of CreateSubViews ().
Terminal.Gui/Views/Selectors/OptionSelector.cs	Extends UpdateChecked () to potentially reset Value if it’s not represented by any checkbox.
Tests/UnitTestsParallelizable/Views/SelectorBaseTests.cs	Adds a test asserting Value resets when swapping Values to a set that doesn’t include the current value.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

[rcdailey]

Looks great. Thank you for fixing this!

[BDisp]

@tig, sometimes the solution takes an immense amount of time to build, constantly processing the GitVersion indefinitely. Often I have to abort the rebuild and try again, but it's really very painful.