Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add copy to clipboard full response functionality in AssistantMessage… #6035

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add copy to clipboard full response functionality in AssistantMessage… #6035

wants to merge 3 commits into from

Commits on Oct 30, 2024

  1. Add copy to clipboard full response functionality in AssistantMessage… 

    …Cell
    @cellamo
    cellamo committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    d65faec View commit details
    Browse the repository at this point in the history

Commits on Oct 31, 2024

  1. refactor: improve copy button implementation in AssistantMessageCell 

    - Replace dangerouslySetInnerHTML with proper React SVG component
- Add accessibility attributes to copy button
- Fix duplicate FeedbackButtons rendering
- Maintain existing styling and telemetry
    @cellamo
    cellamo committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    79adb49 View commit details
    Browse the repository at this point in the history

Commits on Nov 1, 2024

  1. refactor: make copy button always visible in assistant messages 

    - Moved copy button outside of loading/error conditions
- Improved UI layout with consistent button placement
- Added wrapper div for better structure
- Simplified conditional rendering logic
- Maintained existing feedback and context focus functionality
    @cellamo
    cellamo committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    73bda29 View commit details
    Browse the repository at this point in the history