Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new copilot-deprecated variant to preserve the legacy Copilot styling while updating the copilot variant with new design specifications. The implementation adds support for both variants across the codebase, including schema updates, CSS styling, and comprehensive test coverage.
Key Changes:
- Added
copilot-deprecatedvariant to the variant schema alongside existingcopilotvariant - Split CSS styles to maintain legacy
copilot-deprecatedstyles while introducing updatedcopilotstyles - Updated test configurations to use
copilot-deprecatedas the default comparison variant
Reviewed Changes
Copilot reviewed 37 out of 169 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
packages/fluent-theme/src/private/VariantComposer.ts |
Added copilot-deprecated to the variant schema |
packages/test/page-object/src/globals/renderWebChat/index.js |
Added logic to detect and handle both copilot variants |
packages/fluent-theme/src/components/theme/Theme.module.css |
Split styles for copilot and copilot-deprecated variants with distinct visual specifications |
packages/fluent-theme/src/components/sendBox/SendBox.module.css |
Applied shared text counter visibility styles to both variants |
packages/fluent-theme/src/components/activity/PartGroupingDecorator.tsx |
Updated header rendering logic to use copilot-deprecated |
packages/fluent-theme/src/components/activity/PartGroupingDecorator.module.css |
Restructured styles with shared, deprecated-specific, and new copilot-specific sections |
packages/fluent-theme/src/components/activity/ActivityLoader.module.css |
Applied shared loader margin styles to both variants |
__tests__/html2/side-by-side/index.html |
Changed default comparison variant from copilot to copilot-deprecated |
| Multiple test HTML files | Added new test files for copilot-deprecated variant scenarios |
| Multiple snapshot PNG files | Added visual regression test snapshots for the updated copilot variant |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
compulim
approved these changes
Nov 7, 2025
This was referenced Nov 17, 2025
11 tasks
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.
Changelog Entry
copilot-deprecatedfor migrationDescription
This is our first step for the modern copilot Web Chat UX. The PR re-introduces the
copilotvariant and makes old UX available undercopilot-deprecatedvariant temporarily.This covers an initial portion of changes. More to come in subsequent PRs.
Specific Changes
*.copilot-deprecated.*copilottocopilot-deprecated-
CHANGELOG.mdI have updated documentationReview Checklist
z-index)package.jsonandpackage-lock.jsonreviewed