Skip to content

Conversation

@micahmo
Copy link
Member

@micahmo micahmo commented Mar 5, 2025

This PR fixes an issue where selecting text to quote doesn't work if there is other text already in the comment.

Fixes: thunder-app/thunder#1726

Will need a corresponding change in Thunder to update the package.

Here I demo that the problem is fixed, as well as a regression test on other types of block quotes.

qemu-system-x86_64_SaqXLYewAt.mp4

@micahmo micahmo requested a review from hjiangsu March 5, 2025 15:19
Copy link
Member

@hjiangsu hjiangsu left a comment

Choose a reason for hiding this comment

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

LGTM!

@hjiangsu hjiangsu merged commit 34db147 into thunder-app:master Mar 5, 2025
1 check passed
@micahmo micahmo deleted the fix/quote-alternate-data branch March 5, 2025 16:23
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.

Quoting parent comment text by using selection + quote button only works if comment box was empty.

2 participants