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

一键将多个连续文本转换为标签 #6471

Closed
wants to merge 1 commit into from

Conversation

iamqiz
Copy link
Contributor

@iamqiz iamqiz commented Nov 3, 2022

  • Please commit to the dev branch 请提交到 dev 开发分支

详见
https://ld246.com/article/1667438445275

@88250
Copy link
Member

88250 commented Nov 3, 2022

抱歉抱歉,这个功能不考虑合并了,主要原因是不确定使用场景,并且缺乏帮助文档和快捷键设置等相关描述。

@88250 88250 closed this Nov 3, 2022
@iamqiz
Copy link
Contributor Author

iamqiz commented Nov 3, 2022

使用场景不是在文章里说了吗? 8人支持,0人反对
https://ld246.com/article/1667438445275
PR时还需要把帮助文档也提供吗? 谁有那闲工夫

@88250
Copy link
Member

88250 commented Nov 3, 2022

  • 如果不是很多人需要的话我们还是保守一点比较好
  • 帮助文档如果不提供的话没有人知道使用方式哦,另外快捷键设置好像也没有修改,这样的话缺失还比较多,所以无法合并到产品中

@88250
Copy link
Member

88250 commented Nov 3, 2022

PR时还需要把帮助文档也提供吗? 谁有那闲工夫

如果没有闲工夫的话建议不要 PR,因为缺失相关使用文档,这会对用户使用和开发者后期维护造成较大困扰,感谢理解和支持。

@iamqiz
Copy link
Contributor Author

iamqiz commented Nov 3, 2022

关键这个功能默认情况下是不占用任何快捷键的,对小白是没有负面影响的
新版本发布时,用户能根据change log找到这个issue和链滴那篇文章,自然就懂了
如果PR成功的话,我也会修改一下PR描述,介绍一下使用场景方法
之前的PR :
增加水平分隔线快捷键 #6179
我在开头也把功能使用方法描述很清楚了
但还是被否了,
原因是我没设置快捷键, 我也解释了我没用快捷键设置的原因:
Snipaste_2022-11-04_00-26-16
你们回复我了吗?
而且这些PR不是可以被开发者修改之后再进行合并吗?

@88250
Copy link
Member

88250 commented Nov 3, 2022

没有回复可能是看漏了,抱歉。Vanessa 几乎不看 at 的,因为她的 GitHub 通知太多已经完全看不过来了,一些 issue 讨论是我看到以后手动发给她的。

对于 6179 中的 key 加载是不是没有刷新前端?这个是刷新界面请求的 http://127.0.0.1:6806/appearance/langs/zh_CN.json

回到这个问题上,我们对于编辑器方面的 PR 比较慎重,可以说这部分不建议其他开发者来修改,因为坑实在太多。就算只改一小部分,很有可能会导致其他地方出问题,而且编辑器的一些需求无法通过社区投票就确定是否加入,投票只能作为一个参考,并不是真正能够确定需求和开发上线的标准。

就这么说吧,如果一个有点用并且没有副作用的功能以社区投票为准来指导开发的话,那么几乎不会有用户投反对票,因为对于用户来说功能肯定是越多越好。但是对于产品本身而言,堆积功能往往意味着这个产品已经失控。

88250 added a commit that referenced this pull request Nov 3, 2022
88250 added a commit that referenced this pull request Nov 3, 2022
@iamqiz iamqiz deleted the dev-22-11-3-一键打标签 branch November 4, 2022 02:54
@iamqiz
Copy link
Contributor Author

iamqiz commented Nov 4, 2022

好用的功能不是应该越多越好吗? 思源用户不喜欢all in one吗? 思源如果像typora那样功能简单,还会有很多人用吗?

而且我pr的这些代码做的是加法,不修改原有编辑器的功能,用户如果不去主动设置快捷键的话,根本就触发不了那个代码块,因而也不会造成bug什么的, 这就类似插件,只会影响想用的人,而不是所有人
但是插件还不知道什么时候能出,有插件的话我就写插件去了而不会PR,

我现在是不是应该只提议,不PR?以防给编辑器制造bug?
提议的内容是不是应该是那些不会造成堆积功能的功能?

@88250
Copy link
Member

88250 commented Nov 4, 2022

我现在是不是应该只提议,不PR?

可以 PR 的,但是添加功能需要带上用户文档和完善相关设置。

提议的内容是不是应该是那些不会造成堆积功能的功能?

也可以提交新特性请求,但是现阶段基本不会考虑添加新特性了。

@siyuan-note siyuan-note locked as too heated and limited conversation to collaborators Nov 4, 2022
@88250
Copy link
Member

88250 commented Nov 4, 2022

我锁定这个话题了,如果需要继续贡献该特性请重新提交新的 PR,我们会仔细审核代码,谢谢。

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

Successfully merging this pull request may close these issues.

None yet

2 participants