Skip to content

Conversation

@sarahentzel
Copy link
Collaborator

…e of recorder

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 pull request reorganizes the comment recording interface to improve usability by moving the record button, save, and cancel controls to the right side of the audio recorder in comments. The implementation introduces a new reusable RecordButton component and adds a keepItSmall prop to enable compact layouts.

Key Changes:

  • Created a new RecordButton component with support for both small and large sizes, replacing the previous inline button implementation
  • Added keepItSmall prop threading through MediaRecord and WSAudioPlayer to enable compact layouts
  • Restructured CommentEditor layout to position controls (record button, save, cancel) to the right of the audio waveform when recording

Reviewed changes

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

Show a summary per file
File Description
src/renderer/src/control/index.ts Exports the new RecordButton component
src/renderer/src/control/RecordButton.tsx New component providing a reusable record button with accessibility features and size variants
src/renderer/src/components/WSAudioPlayer.tsx Adds keepItSmall prop support, conditionally renders RecordButton based on layout mode, removes unused Button and icon imports
src/renderer/src/components/MediaRecord.tsx Threads keepItSmall prop from CommentEditor to WSAudioPlayer
src/renderer/src/components/Discussions/CommentEditor.tsx Restructures layout to display save/cancel buttons and recorder side-by-side, removes unnecessary eslint-disable comment

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

@sarahentzel sarahentzel requested a review from gtryus December 13, 2025 00:05
@gtryus gtryus merged commit 331b1a9 into develop Dec 13, 2025
1 check passed
@gtryus gtryus deleted the TT-6983 branch December 13, 2025 22:26
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