Skip to content

Replace comment icon with quote icon for quote reply functionality#7188

Merged
alexr00 merged 3 commits intomainfrom
copilot/fix-7185
Jul 2, 2025
Merged

Replace comment icon with quote icon for quote reply functionality#7188
alexr00 merged 3 commits intomainfrom
copilot/fix-7185

Conversation

Copy link
Contributor

Copilot AI commented Jul 2, 2025

The quote reply functionality was using a comment icon (speech bubble) instead of a proper quote icon, which was confusing to users who expected to see a quote symbol for quoting functionality.

Changes Made

  • Added quote icon: Copied the VS Code quote.svg codicon to resources/icons/
  • Updated icon exports: Added quoteIcon export in webviews/components/icon.tsx
  • Replaced icon usage: Updated the quote reply button in webviews/components/comment.tsx to use quoteIcon instead of commentIcon

Visual Impact

Before: Speech bubble icon (💬) for "Quote reply" - ambiguous and could be confused with adding a new comment

After: Double quotes icon (❝❞) for "Quote reply" - intuitive and matches user expectations

This change aligns the extension's UI with conventions used by other platforms like Microsoft Teams and GitHub.com, making the quote functionality more discoverable and intuitive for users.

Testing

  • ✅ Webviews compile successfully without errors
  • ✅ No breaking changes to existing functionality
  • ✅ Icon displays correctly in the comment action bar

Fixes #7185.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Copilot AI changed the title [WIP] Don't use comment icon to mean quote Replace comment icon with quote icon for quote reply functionality Jul 2, 2025
Copilot AI requested a review from alexr00 July 2, 2025 08:49
@alexr00 alexr00 marked this pull request as ready for review July 2, 2025 09:37
@vs-code-engineering vs-code-engineering bot added this to the June 2025 milestone Jul 2, 2025
@alexr00 alexr00 merged commit b93010d into main Jul 2, 2025
3 checks passed
@alexr00 alexr00 deleted the copilot/fix-7185 branch July 2, 2025 10:45
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.

Don't use comment icon to mean quote

4 participants