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

面包屑改进:Click 为聚焦,聚焦返回文案修改 #4916

Closed
royc01 opened this issue May 16, 2022 · 19 comments
Closed

面包屑改进:Click 为聚焦,聚焦返回文案修改 #4916

royc01 opened this issue May 16, 2022 · 19 comments
Assignees
Milestone

Comments

@royc01
Copy link

royc01 commented May 16, 2022

222

@Vanessa219
Copy link
Member

这个是考虑文档已经加载过了,只要定位下就行,省的用户点来点去的。在实际使用过程中,你觉得是定位好呢,还是聚焦比较好?

@royc01
Copy link
Author

royc01 commented May 16, 2022

明白了,定位也很实用,但在这个基础上可以加入跳转功能么?类似浮窗面包屑ctrl+点击的功能

@fanglypro
Copy link

fanglypro commented May 17, 2022

这个是考虑文档已经加载过了,只要定位下就行,省的用户点来点去的。在实际使用过程中,你觉得是定位好呢,还是聚焦比较好?

这两者不冲突呀,鼠标放上去不点击的时候是定位,点击是聚焦,以前版本不是都这么设计的吗?

这看起来是个bug,当我随便聚焦到文档内的一个块的时候,一切正常,点击后能够聚焦,而当我聚焦到文档块的时候,点不动:

temp237

@88250 88250 modified the milestones: 2.0.11, 2.0.12 May 19, 2022
@tcmtom
Copy link

tcmtom commented May 19, 2022

支持点击后聚焦

@Vanessa219
Copy link
Member

@royc01 @tcmtom 那点击就修改为进行聚焦吧
@fanglypro 以前的版本是鼠标移上去高亮,点击没有对内容是否加载进行业务的细化。这不是一个bug,是特性哦。

@fanglypro
Copy link

fanglypro commented May 19, 2022

@royc01 @tcmtom 那点击就修改为进行聚焦吧 @fanglypro 以前的版本是鼠标移上去高亮,点击没有对内容是否加载进行业务的细化。这不是一个bug,是特性哦。

我试了2.0.6,是有这个功能的,无论当前聚焦在文档块还是非文档块,鼠标点击后都可以聚焦,应该是2.0.6之后的某个改动导致这个功能失效了,我以前经常用这个功能的,应该说的是同一个功能吧:joy:

temp238

@winter60
Copy link

@Vanessa219 其实聚焦不聚焦对我来说还好,主要是想通过面包屑能够定位块。目前好像只能高亮,进入聚焦块后面包屑才能定位返回上级,回到非聚焦块状态就不能定位了。

@Zuoqiu-Yingyi
Copy link
Contributor

现在点击后不能聚焦嘛,我这里怎么可以呀❓

@CleinBlue
Copy link

我个人倒是觉得维持现有的逻辑不错,即点击高亮。因为有时不需要聚焦,可以设置为某个按键+点击=聚焦吗?,也希望有更多意见。

@winter60
Copy link

我个人倒是觉得维持现有的逻辑不错,即点击高亮。因为有时不需要聚焦,可以设置为某个按键+点击=聚焦吗?,也希望有更多意见。

妙啊,和原来风格一样了。移动到面包屑时为高亮,点击时为定位,ctrl+点击为聚焦块。

@Vanessa219
Copy link
Member

@fanglypro 对,前面说过了,加了一个特性:内容块在当前编辑器中就进行高亮,否则才进行缩放。

@Vanessa219 Vanessa219 changed the title 点击面包屑不能进入聚焦 ctrl+点击面包屑进入聚焦 May 20, 2022
@Vanessa219
Copy link
Member

@winter60 定位在这个版本中再添加一下。

@Zuoqiu-Yingyi 因为你点击的面包屑的块没有显示在文档中

@Vanessa219 Vanessa219 changed the title ctrl+点击面包屑进入聚焦 面包屑改进:点击为高亮并定位,ctrl点击为聚焦 May 20, 2022
@88250 88250 changed the title 面包屑改进:点击为高亮并定位,ctrl点击为聚焦 面包屑改进:Click 为高亮并定位,Ctrl+Click 为聚焦 May 20, 2022
@fanglypro
Copy link

@fanglypro 对,前面说过了,加了一个特性:内容块在当前编辑器中就进行高亮,否则才进行缩放。

但是现在这个版本中,内容块在当前编辑器中也可能会缩放,比如下面这个动图中,6这个列表项已经加载了,我点击后依然会聚焦。

temp239

编辑器聚焦在文档块和非文档块时的面包屑的交互逻辑是不同,逻辑上感觉有点混乱。

貌似是这个新特性只对当前聚焦在文档块时有效,但对当前聚焦在非文档块时并不有效,两者没有统一,这个issue会对两者的交互逻辑进行统一么?

@Zuoqiu-Yingyi
Copy link
Contributor

Zuoqiu-Yingyi commented May 20, 2022

@Vanessa219 我认为文档面包屑与悬浮窗面包屑的行为应该统一, click 在悬浮窗面包屑的行为是聚焦, Ctrl + click 在悬浮窗面包屑的行为是跳转, 那么文档的面包屑也应保持一致, 同时对用户原来的习惯影响也较小, 因此我建议

  • Click 为聚焦
  • Ctrl+Click 为高亮并定位(也就是跳转)

@Vanessa219
Copy link
Member

Vanessa219 commented May 20, 2022

@Zuoqiu-Yingyi 悬浮窗的面包屑需要展开为页签,因此 Ctrl+Click 是打开新的页签并聚焦,对于页签也说也就是聚焦,加之块标的聚焦也是 ctrl +click。因此还是按照楼上的建议进行修改。

@fanglypro 这个应该是 bug,会统一逻辑的。

@fanglypro
Copy link

fanglypro commented May 20, 2022

@Zuoqiu-Yingyi 悬浮窗的面包屑需要展开为页签,因此 Ctrl+Click 是打开新的页签并聚焦,对于页签也说也就是聚焦,加之块标的聚焦也是 ctrl +click。因此还是按照楼上的建议进行修改。

@fanglypro 这个应该是 bug,会统一逻辑的。

现在悬浮窗的面包屑在 Ctrl+Click 的时候并不一定是打开新的页签并聚焦,也可能是打开新的页签并跳转(高亮并定位),如图:

temp241

这部分的逻辑有点混乱,我建议面包屑上的click功能统一按照大纲软件的普遍用法,设置为聚焦,用户的心智负担更小,在所有场景下面包屑click的含义都是相同,ctrl+click功能可以设置为其他思源特色的功能。

目前issue中所写的交互逻辑会导致面包屑的click和ctrl+click在不同场景下两者都有不同的含义,而 @Zuoqiu-Yingyi 所提的逻辑至少能保证面包屑click的含义在不同场景下是相同的,降低用户的心智负担。

@Vanessa219
Copy link
Member

Vanessa219 commented May 20, 2022

好的,那就还是和以前保持一致。
还想再请教一下聚焦返回。是应该返回当前块的父亲层,还是返回当前面包屑的上一层呢?比如下图,列表项4的聚焦返回到列表项2还是列表项 f1d 上要好一点呢?
image

@fanglypro
Copy link

fanglypro commented May 20, 2022

好的,那就还是和以前保持一致。 还想再请教一下聚焦返回。是应该返回当前块的父亲层,还是返回当前面包屑的上一层呢?比如下图,列表项4的聚焦返回到列表项2还是列表项 f1d 上要好一点呢? image

聚焦返回是指alt+←

这个聚焦返回是和聚焦、复制、折叠展开这些功能并列的,那么应该是块级别的,应当以该块为中心进行操作,所以,列表项4的聚焦返回在逻辑上应该是到列表项2。

“返回当前面包屑的上一层”这是文档级别的操作了,如果要实现这个功能的快捷键,我认为应当单独再设置一个全局快捷键,而不是占用针对某个块的快捷键。也就是说这应该实现成两个功能,一个是“聚焦到父块”(会显示在块标的列表中),另一个是“聚焦到当前面包屑的上一层”(不会显示在块标的列表中),应该分别设置快捷键。

“聚焦返回”这个词也有一点奇怪,我刚开始还以为是ctrl+[这个后退功能在针对聚焦时的特例,或许把描述文本改成“聚焦到父块”之类更具体的表述更合适一点。

从应用上来讲,我感觉聚焦到父块这个功能用处不大,“返回当前面包屑的上一层”这个功能用得会更多

@Vanessa219
Copy link
Member

面包屑还需要改进成 IDE 这样可用鼠标/键盘操作的。目前就先把文案改为“聚焦到上级”吧,谢谢。
image

@Vanessa219 Vanessa219 changed the title 面包屑改进:Click 为高亮并定位,Ctrl+Click 为聚焦 面包屑改进:Click 为聚焦,聚焦返回文案修改 May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants