Skip to content

Conversation

@BorisGerretzen
Copy link
Owner

  • Added Blazor server demo
  • Moved demo pages into razor classlib
  • Added projects to solution folders.

Copilot AI review requested due to automatic review settings November 16, 2025 12:41
@BorisGerretzen BorisGerretzen merged commit 7ab2e18 into master Nov 16, 2025
6 checks passed
@BorisGerretzen BorisGerretzen deleted the chore/refactor-demo branch November 16, 2025 12:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

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