Skip to content

Conversation

PhotonicDev
Copy link

Success criteria

Please describe what should be possible after this change. List all individual items on a separate line.

  • A Should be able to accept the new icons and animations and display them accordingly
  • B
  • C

How to test

Please describe the individual steps on how a peer can test your change.

  1. A you should change the parameters in the dist/index.html to include in "config.layout.iconUrl" "config.layout.iconAnimation", "config.layout.iconAnimationSpeed", "config.layout.iconAnimationInterval".
  2. B adjust them to the appropriate values and test it out, or try to setup the proper endpoint config and test it out all together.
  3. C

Security

  • Possible injection vector
  • Authentication/Access controls touched
  • Sensitive Data could be exposed
  • XSS
  • Logging/Monitoring touched
  • Exchanges data with external systems
  • No security implications

Additional considerations

  • This PR might have performance implications

Documentation Considerations

These are hints for the documentation team to help write the docs.

@Copilot Copilot AI review requested due to automatic review settings October 15, 2025 15:53
Copy link
Contributor

@Copilot 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 implements chat bubble setup for webchat by adding support for customizable icons and animations on the webchat toggle button (FAB). The changes enable configuration of icon URLs, animation types, speeds, and intervals through the webchat config.

  • Adds animation support (bounce, swing, pulse) with CSS keyframes for the chat toggle button
  • Implements configurable icon system supporting default icons, SVG files, and data URIs
  • Introduces interval-based animation triggering with configurable timing

Reviewed Changes

Copilot reviewed 4 out of 7 changed files in this pull request and generated 2 comments.

File Description
src/webchat-ui/components/presentational/FAB.tsx Adds CSS keyframes and animation classes for bounce, swing, and pulse effects
src/webchat-ui/components/presentational/FABDisabled.tsx Adds hover/active state styling for disabled FAB
src/webchat-ui/components/WebchatUI.tsx Implements icon selection logic, animation interval management, and renders configurable chat icons
src/common/interfaces/webchat-config.ts Extends layout settings interface with new icon and animation configuration options

Tip: Customize your code reviews with copilot-instructions.md. Create the file or 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.

1 participant