Skip to content

[Convai Widget] Fix button cursor style#896

Merged
giannagerton merged 2 commits into
mainfrom
aarkalyk/AP-2416-fix-widget-button-hover-styles
Jul 22, 2026
Merged

[Convai Widget] Fix button cursor style#896
giannagerton merged 2 commits into
mainfrom
aarkalyk/AP-2416-fix-widget-button-hover-styles

Conversation

@aarkalyk

@aarkalyk aarkalyk commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Fixes the styling on the Convai Widget buttons.

Screenshots

Before After
Screenshot 2026-07-22 at 10 31 30 Screenshot 2026-07-22 at 10 31 11

Note

Low Risk
Cosmetic CSS only; no auth, data, or logic changes.

Overview
Convai Widget buttons now show a pointer cursor on hover by setting cursor: pointer on button elements inside the widget host styles (index.css).

Includes a minor indentation-only change in Button.tsx (no behavior change).

Reviewed by Cursor Bugbot for commit 6621568. Bugbot is set up for automated code reviews on this repo. Configure here.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Convai Widget’s button styling so interactive buttons consistently show the pointer cursor, aligning the UI affordance with the actual click behavior.

Changes:

  • Add cursor-pointer to the core Button component base classes.
  • Add cursor-pointer to the widget expand button.
  • Add cursor-pointer to the pending file remove (“x”) button.

Reviewed changes

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

File Description
packages/convai-widget-core/src/widget/PendingFilePreview.tsx Ensures the remove-file icon button displays a pointer cursor on hover.
packages/convai-widget-core/src/components/ExpandButton.tsx Ensures the expand/open-chat button displays a pointer cursor on hover.
packages/convai-widget-core/src/components/Button.tsx Makes the shared Button component display a pointer cursor by default for interactive states.

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

@aarkalyk
aarkalyk force-pushed the aarkalyk/AP-2416-fix-widget-button-hover-styles branch from ed109bf to 23c0422 Compare July 22, 2026 08:45
@giannagerton

Copy link
Copy Markdown

@aarkalyk Maybe instead of adding cursor-pointer to each button, we could set it once globally on the button

@aarkalyk
aarkalyk force-pushed the aarkalyk/AP-2416-fix-widget-button-hover-styles branch from 847a3a3 to 6621568 Compare July 22, 2026 16:30
@aarkalyk
aarkalyk requested a review from giannagerton July 22, 2026 16:56
@giannagerton
giannagerton merged commit d9ef175 into main Jul 22, 2026
5 checks passed
@giannagerton
giannagerton deleted the aarkalyk/AP-2416-fix-widget-button-hover-styles branch July 22, 2026 17:46
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