Skip to content

Conversation

@valkirilov
Copy link
Member

@valkirilov valkirilov commented Nov 26, 2025

What

Migrated CopilotTrigger component from SCSS modules to styled-components following the project's migration guidelines.

Testing

  1. Verify Copilot trigger button appears correctly
  2. Click to open/close panel and verify active state styling
  3. Verify tooltip appears on hover
Before After
image image
image image

The Insights icon remains the same

Light Dark
image image

Closes #RI-7774


Note

Migrates Copilot trigger from SCSS to styled-components and updates Insights trigger to use themed styling and cleaned imports.

  • UI:
    • Copilot Trigger (copilot-trigger):
      • Migrate from SCSS modules to styled-components (CopilotTrigger.styles.ts).
      • Introduce CopilotWrapper and CopilotIconButton with isOpen state-driven background.
      • Set icon size via SVG and use size="S", tooltip, and ARIA; remove styles.module.scss.
    • Insights Trigger (insights-trigger):
      • Use theme color for BulbHighlighting and simplify isOpen styling.
      • Normalize styles filename import casing and remove unused css import.

Written by Cursor Bugbot for commit 2ba4069. This will update automatically on new commits. Configure here.

…onents

Migrated CopilotTrigger component from SCSS modules to styled-components
following the project's migration guidelines. Replaced styles.module.scss
with CopilotTrigger.styles.ts using FlexGroup and IconButton styled components.
Removed classnames dependency and improved type safety.

References: #RI-7774
Changed the icon color in CopilotTrigger.styles.ts to utilize the theme's attention color for better consistency with the overall design system.
Removed unnecessary imports of 'css' from styled-components in CopilotTrigger and insights-trigger styles. Updated the conditional styling for the background color to use a more concise syntax, enhancing readability and maintainability.
…iable

Changed the background color in insights-trigger.styles.ts to utilize the theme's attention color for improved consistency with the design system.
@github-actions
Copy link
Contributor

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.9% 20906/25218
🟡 Branches 68.06% 8817/12954
🟡 Functions 77.96% 5714/7329
🟢 Lines 83.32% 20473/24572

Test suite run success

5411 tests passing in 702 suites.

Report generated by 🧪jest coverage report action from 2ba4069

jit-ci[bot]

This comment was marked as outdated.

@valkirilov valkirilov merged commit d8879ef into main Nov 27, 2025
19 checks passed
@valkirilov valkirilov deleted the fe/bugfix/RI-7774/copilot-button branch November 27, 2025 13:47
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.

4 participants