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.

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

fix: issues when replying to comments #10912

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nickoferrall
Copy link
Contributor

Fix #10223

Demo: https://www.loom.com/share/971e6d96cdd947eb9629fdc7c9672914

This fixes three issues:

  1. When replying to yourself, initially, it doesn't prefix the reply with the user's name. Only when replying to a reply does it do this.
  2. When replying to yourself, it shouldn't add your name as the prefix.
  3. When Bob replies to Susan, prefix Bob's comment with @Susan and ensure the cursor starts after @Susan. Currently, it's before.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Discussions: when replying to self, do not start with @self
1 participant