-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
一键将多个连续文本转换为标签 #6471
The head ref may contain hidden characters: "dev-22-11-3-\u4E00\u952E\u6253\u6807\u7B7E"
一键将多个连续文本转换为标签 #6471
Conversation
抱歉抱歉,这个功能不考虑合并了,主要原因是不确定使用场景,并且缺乏帮助文档和快捷键设置等相关描述。 |
使用场景不是在文章里说了吗? 8人支持,0人反对 |
|
如果没有闲工夫的话建议不要 PR,因为缺失相关使用文档,这会对用户使用和开发者后期维护造成较大困扰,感谢理解和支持。 |
关键这个功能默认情况下是不占用任何快捷键的,对小白是没有负面影响的 |
没有回复可能是看漏了,抱歉。Vanessa 几乎不看 at 的,因为她的 GitHub 通知太多已经完全看不过来了,一些 issue 讨论是我看到以后手动发给她的。 对于 6179 中的 key 加载是不是没有刷新前端?这个是刷新界面请求的 回到这个问题上,我们对于编辑器方面的 PR 比较慎重,可以说这部分不建议其他开发者来修改,因为坑实在太多。就算只改一小部分,很有可能会导致其他地方出问题,而且编辑器的一些需求无法通过社区投票就确定是否加入,投票只能作为一个参考,并不是真正能够确定需求和开发上线的标准。 就这么说吧,如果一个有点用并且没有副作用的功能以社区投票为准来指导开发的话,那么几乎不会有用户投反对票,因为对于用户来说功能肯定是越多越好。但是对于产品本身而言,堆积功能往往意味着这个产品已经失控。 |
好用的功能不是应该越多越好吗? 思源用户不喜欢all in one吗? 思源如果像typora那样功能简单,还会有很多人用吗? 而且我pr的这些代码做的是加法,不修改原有编辑器的功能,用户如果不去主动设置快捷键的话,根本就触发不了那个代码块,因而也不会造成bug什么的, 这就类似插件,只会影响想用的人,而不是所有人 我现在是不是应该只提议,不PR?以防给编辑器制造bug? |
可以 PR 的,但是添加功能需要带上用户文档和完善相关设置。
也可以提交新特性请求,但是现阶段基本不会考虑添加新特性了。 |
我锁定这个话题了,如果需要继续贡献该特性请重新提交新的 PR,我们会仔细审核代码,谢谢。 |
详见
https://ld246.com/article/1667438445275