Merged
Conversation
Owner
BorisGerretzen
commented
Nov 16, 2025
BorisGerretzen
commented
- Added Blazor server demo
- Moved demo pages into razor classlib
- Added projects to solution folders.
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the demo project structure by extracting shared demo pages into a Razor class library (SimpleBlazorMultiselect.Demo.Shared) and adds a new Blazor Server demo alongside the existing WebAssembly demo. The solution is also reorganized with folder structure for better project management.
Key changes:
- Comprehensive test suite added with multiple test categories (styling, equality, disabled states, etc.)
- Demo pages moved to a shared Razor class library for reuse between WASM and Server demos
- Build scripts updated to work with new solution structure
Reviewed Changes
Copilot reviewed 27 out of 63 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/SimpleBlazorMultiselect.Tests/* | New comprehensive test suite with helper classes and multiple test categories |
| demo/SimpleBlazorMultiselect.Demo.Shared/* | New shared Razor class library containing reusable demo pages and layouts |
| demo/SimpleBlazorMultiselect.Demo.Wasm/* | Refactored WebAssembly demo to use shared library |
| demo/SimpleBlazorMultiselect.Demo.Server/* | New Blazor Server demo project |
| build/Build.cs | Updated to use GetAllProjects for compatibility with solution folders |
| SimpleBlazorMultiselect.sln | Reorganized with solution folders (src, tests, demo) |
| README.md | Updated with reference to demo folder location |
| .gitignore | Added .idea exclusions |
Comments suppressed due to low confidence (1)
.vs/SimpleBlazorMultiselect/v17/DocumentLayout.json:1
- Visual Studio metadata files (.vs directory contents) should not be committed to the repository. These are IDE-specific files that should be excluded via .gitignore. The .gitignore already includes
.vs/and.vs, so these files should be removed from the repository.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.