Skip to content

Conversation

@OEvgeny
Copy link
Collaborator

@OEvgeny OEvgeny commented Nov 17, 2025

Changelog Entry

  • (Experimental) Reworked the copilot variant to align with the modern Copilot UX, in PR #5630, by @OEvgeny, in PR #5634, by @OEvgeny
    • The legacy design is temporarily available as copilot-deprecated for migration
    • Added loading animation for copilot, copilot-deprecated and fluent variants

Description

This continues the work on the new copilot variant and fixes some issues found in white-label along the way.

The new loader animation for fluent-theme package was added as well as the styling for grouping was improved.

Specific Changes

-

  • I have added tests and executed them locally
  • I have updated CHANGELOG.md
  • I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • Browser and platform compatibilities reviewed
  • CSS styles reviewed (minimal rules, no z-index)
  • Documents reviewed (docs, samples, live demo)
  • Internationalization reviewed (strings, unit formatting)
  • package.json and package-lock.json reviewed
  • Security reviewed (no data URIs, check for nonce leak)
  • Tests reviewed (coverage, legitimacy)

@OEvgeny OEvgeny requested a review from Copilot November 17, 2025 22:37
Copy link
Contributor

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 updates the Copilot variant to align with modern Copilot UX design and adds loading animations for multiple variants. The changes primarily involve standardizing the capitalization of creativeWorkStatus values from lowercase to title case (e.g., 'incomplete' to 'Incomplete') to match current API conventions.

Key Changes:

  • Standardized creativeWorkStatus property values to use title case capitalization
  • Updated documentation to reflect the new capitalized status values
  • Added loading animation support for copilot, copilot-deprecated, and fluent variants

Reviewed Changes

Copilot reviewed 2 out of 647 changed files in this pull request and generated no comments.

File Description
__tests__/html2/activity/message-status.html Updated test cases to use title case for creativeWorkStatus values ('Incomplete' and 'Published')
CHANGELOG.md Updated documentation to reflect the capitalized status values and added note about loading animations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OEvgeny OEvgeny marked this pull request as ready for review November 17, 2025 22:59
@OEvgeny
Copy link
Collaborator Author

OEvgeny commented Nov 17, 2025

For the fluent-theme the chevron angle is consistent with our Figma:

  • Closed state pointing to the right
  • Opened state pointing to the bottom

This is consistent with say codespaces/vscode:
image

But not with fluentai ChainOfThought implementation:
image

@OEvgeny OEvgeny requested a review from compulim November 18, 2025 02:23
@OEvgeny OEvgeny merged commit c8aae90 into main Nov 18, 2025
29 checks passed
@OEvgeny OEvgeny deleted the feat/copilot-cot-v1 branch November 18, 2025 02:29
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.

3 participants