维护者确认:PR #3925 成对符号包裹选中文本
- Dominant language
- TypeScript
- Stars
- 2.7k
- Forks
- 401
- Avg merge
- 21h 48m
- Merged PRs (30d)
- 776
Description
## 为什么需要维护者确认
PR https://github.com/makecindy/cindy/pull/3925(`feat(desktop): 支持成对符号包裹选中文本`)是新的产品交互,不是既有功能的补丁或纯技术改动。
作者在主聊天输入、共享 textarea、以及纯文本 / Markdown / 代码的 CodeMirror 编辑器里,加入了类似 VS Code 的选区包裹:选中文本后再输入 `"`、`'`、`(`、`[`、`{`、`<` 时,原文会被成对符号包住而不是被覆盖。
这会改变用户每天都在用的输入手感,并覆盖多个编辑器入口。需要维护者确认:
- 这个交互是否应该成为 Cindy 的默认行为(目前没有开关)
- 覆盖范围(主聊天、消息编辑、文件编辑器)是否合适
- 空选区不自动补齐、输入右符号不跳过,是否符合产品预期
## 怎么确认
- 同意:直接在 PR 上 Approve。Approve 即放行维护者确认门。
- 需要修改:在 PR 上 Request Changes,并写明要改什么。作者改完后,再重新 Approve 即放行。
- 不要只在评论或本 issue 里口头说「可以」——那不会放行。
关联 PR:https://github.com/makecindy/cindy/pull/3925
---
关联 PR:#3925(作者 @Stella-xixi);本 issue 由 review-pr 流程自动创建,用于先讨论该 PR 涉及的产品 / UI 变更,维护者确认后 PR 会恢复推进。
Contributor guide
Research direction
Review PR #3925 first, focusing on its selection-wrapping behavior across the main chat input, shared textarea, and CodeMirror editors. Check whether the default interaction and listed edge cases match the product expectations; done means approving the PR or requesting specific changes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100