Skip to content

Conversation

lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Sep 12, 2025

The previous hack for getting the docs to start at the bottom broke selecting text (for copy/paste) across messages. This fix still uses flex-direction: column-reverse to get them to stick to / start at the end, but it puts them all together in one div so that we don't have to reverse the order which breaks selecting text with the mouse across messages.

ie. It is now possible to select and copy a prompt plus the answer.

@lerouxb lerouxb requested a review from a team as a code owner September 12, 2025 10:57
@Copilot Copilot AI review requested due to automatic review settings September 12, 2025 10:57
@lerouxb lerouxb added the no release notes Fix or feature not for release notes label Sep 12, 2025
@lerouxb lerouxb added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Sep 12, 2025
Copy link
Contributor

@Copilot 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 PR fixes a text selection issue in the compass-assistant chat component where users couldn't select text across multiple messages for copy/paste operations. The fix maintains the visual layout of messages starting at the bottom while enabling proper text selection by grouping all messages in a single container instead of reversing their order.

  • Removes message order reversal that was breaking text selection
  • Wraps all messages in a single div container to enable cross-message text selection
  • Updates CSS selector to accommodate the new DOM structure

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added the fix label Sep 12, 2025
Copy link
Contributor

@gagik gagik left a comment

Choose a reason for hiding this comment

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

awesome! much better solution in retrospect

@lerouxb lerouxb removed the request for review from paula-stacho September 12, 2025 11:00
@lerouxb lerouxb merged commit 86d9305 into main Sep 12, 2025
57 of 62 checks passed
@lerouxb lerouxb deleted the fix-select-copy branch September 12, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix no release notes Fix or feature not for release notes no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants