refactor(Select): refactor the editable mode logic

[ArgoZhang]

refactor the editable mode logic

Summary of the changes (Less than 80 chars)

简单描述你更改了什么, 不超过80个字符；如果有关联 Issue 请在下方填写相关编号

Description

fixes #4605

Regression?

• Yes
• No

[If yes, specify the version the behavior has regressed from]

[是否影响老版本]

Risk

• High
• Medium
• Low

[Justify the selection above]

Verification

• Manual (required)
• Automated

Packaging changes reviewed?

• Yes
• No
• N/A

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

• Doc is updated/provided or not needed
• Demo is updated/provided or not needed
• Merge the latest code from the main branch

Summary by Sourcery

Refactor the editable mode logic in the Select component to improve handling of generic types and fix issue #4605 by ensuring proper insertion of new SelectedItems.

Bug Fixes:

• Fix issue refactor(Select): refactor the editable mode logic #4605 by ensuring that a new SelectedItem is inserted into Items when it is not already present.

Enhancements:

• Refactor the logic for determining if Items is a generic type and handling SelectedItem insertion.

[sourcery-ai]

Reviewer's Guide by Sourcery

This PR refactors the editable mode logic in the Select component, specifically improving how generic type handling and value conversion are managed when a new item is added through the editable input.

No diagrams generated as the changes look simple and do not need a visual representation.

File-Level Changes

Change	Details	Files
Improved generic type detection logic for SelectedItem	Moved generic type check inside the item-null condition block Enhanced generic type detection to check if the generic argument itself is generic Relocated TValue default initialization to be only used when needed	src/BootstrapBlazor/Components/Select/Select.razor.cs
Added direct string value assignment for new items	Added CurrentValueAsString assignment when creating a new item Ensures the input value is properly reflected in the component's state	src/BootstrapBlazor/Components/Select/Select.razor.cs

Assessment against linked issues

Issue	Objective	Addressed	Explanation
#4605	Update and refactor the IsEditable logic in the Select component	✅

Possibly linked issues

• refactor(Select): refactor the editable mode logic #4605: The PR refactors the editable mode logic, directly addressing the issue's request to update IsEditable logic.
• refactor(Select): refactor the editable mode logic #4605: PR refactors editable mode logic, addressing the issue of form blocking Select component.
• refactor(Select): refactor the editable mode logic #4605: The PR refactors Select component logic, addressing the issue with OnSelectedItemChanged triggering.

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time. You can also use
  this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[sourcery-ai]

Hey @ArgoZhang - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider adding error handling for TextConvertToValueCallback to gracefully handle conversion failures

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (a802e1a) to head (28d95fe).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #4606   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          620       620           
  Lines        27367     27373    +6     
  Branches      3914      3915    +1     
=========================================
+ Hits         27367     27373    +6     

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