Skip to content

refactor(components):Remove unnecessary nullable string in select ValueChange callback.#261

Merged
TeseySTD merged 1 commit intoRazorConsole:mainfrom
TeseySTD:refactor/select
Dec 21, 2025
Merged

refactor(components):Remove unnecessary nullable string in select ValueChange callback.#261
TeseySTD merged 1 commit intoRazorConsole:mainfrom
TeseySTD:refactor/select

Conversation

@TeseySTD
Copy link
Member

Overview

The Options parameter is explicitly defined as non-nullable string[], so making EventCallback<string?> nullable was redundant.

This change aligns the event signature with the data source, resolving ReSharper warnings about method group variance mismatches (e.g., seen in the Gallery example):

gallery example

@TeseySTD TeseySTD enabled auto-merge (squash) December 21, 2025 10:33
@github-actions
Copy link

🚀 Preview Deployment

A preview build has been generated for this PR from CI run #20408542608!

Download the artifact:
website-preview-261-578381e571501f34ffd1e95affa4c5f5f62b5d04

To view the preview locally:

  1. Download the artifact from the CI workflow run
  2. Extract the ZIP file
  3. Serve the files with a local web server
    (e.g., npx serve dist)

🌐 Live Preview URL: https://2d507fda.razorconsole.pages.dev

The live preview will be automatically updated when you push new
commits to this PR.

@TeseySTD TeseySTD merged commit 3885ad1 into RazorConsole:main Dec 21, 2025
7 checks passed
@github-actions github-actions bot added this to the v0.3.0 milestone Dec 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants