makecindy / makecindy/cindy

维护者确认:PR #3925 成对符号包裹选中文本

Open
#3,927 1 comment 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.