logseq / logseq

A privacy-first, open-source platform for knowledge management and collaboration. Download link: http://github.com/logseq/logseq/releases. roadmap: http://trello.com/b/8txSM12G/roadmap
https://logseq.com
GNU Affero General Public License v3.0
30.26k stars 1.77k forks source link

亮色主题异样! #952

Closed Pathsis closed 3 years ago

Pathsis commented 3 years ago

亮色主题的所有链接文本的颜色都变成灰色的了。

复选框也看不到了。

tiensonqin commented 3 years ago

我看下啊

Pathsis commented 3 years ago

还有状态提示的布局变化了,确切地说,以前挺好,现在变丑了:

截屏2020-12-18 16 37 09

push和pull按钮的大小不一致,且挤在一起。

tiensonqin commented 3 years ago

可能是 css 的 build 有问题,我们在查

Pathsis commented 3 years ago

@tiensonqin te

再反映一个问题,即侧边栏的文本显示宽度大大超过之前的(无论暗黑或亮色都是如此),以下是昨晚和现在的对比。

截屏2020-12-18 01 20 22

截屏2020-12-18 18 05 28

tiensonqin commented 3 years ago

@mkyos 其他问题已经修复,侧边栏宽度目前是自适应的,之前设了最大的固定宽度。这个目前暂不做调整了,后面要做成可以自由拖动的。其他问题确定 ok 的话,我再关掉这个 issue.

Pathsis commented 3 years ago

@mkyos 其他问题已经修复,侧边栏宽度目前是自适应的,之前设了最大的固定宽度。这个目前暂不做调整了,后面要做成可以自由拖动的。其他问题确定 ok 的话,我再关掉这个 issue.

神速!太棒了!惊喜地发现亮色主题下的向右移动问题似乎也修复了,以前都不敢吹毛求疵地请求修复的。特别感谢!😊

Pathsis commented 3 years ago

@mkyos 其他问题已经修复,侧边栏宽度目前是自适应的,之前设了最大的固定宽度。这个目前暂不做调整了,后面要做成可以自由拖动的。其他问题确定 ok 的话,我再关掉这个 issue.

@tiensonqin

刚刚测试了下,发现了一个问题:

侧边栏打开的笔记在滚动时没有滚动条显示进度了,仅仅有对左侧笔记的滚动条指示。

Pathsis commented 3 years ago

@tiensonqin ten

第二个问题,就是在滚动侧边栏页面时,如果滚动到顶或底时,会接着滚动左侧页面。我觉得这个交互是不必要的,以前也不存在这个情况,当然您可能有自己的考虑。

Pathsis commented 3 years ago

@tiensonqin

报告第三个问题,英文断行问题:

截屏2020-12-18 22 51 30

↑在左侧的页面有这一问题

截屏2020-12-18 22 12 59

↑在右侧的页面也有这一问题

Pathsis commented 3 years ago

@tiensonqin

第四个问题,侧边栏打开新的页面,侧边栏不自动滚动到新打开的页面位置。当然,这也不能算是问题,但以前是滚动到新页面的位置,因为按照习惯是即将在新打开的页面操作的。

tiensonqin commented 3 years ago

@mkyos 其他问题已经修复,侧边栏宽度目前是自适应的,之前设了最大的固定宽度。这个目前暂不做调整了,后面要做成可以自由拖动的。其他问题确定 ok 的话,我再关掉这个 issue.

@tiensonqin

刚刚测试了下,发现了一个问题:

侧边栏打开的笔记在滚动时没有滚动条显示进度了,仅仅有对左侧笔记的滚动条指示。

这个之前应该是没有的吧,因为右边两个滚动条的话有点太密集了,这块还在想怎么优化。

tiensonqin commented 3 years ago

@tiensonqin ten

第二个问题,就是在滚动侧边栏页面时,如果滚动到顶或底时,会接着滚动左侧页面。我觉得这个交互是不必要的,以前也不存在这个情况,当然您可能有自己的考虑。

这个可能是最近的修改改变了行为,@xyhp915 你觉得这个交互要不要去掉?

tiensonqin commented 3 years ago

@tiensonqin

报告第三个问题,英文断行问题:

截屏2020-12-18 22 51 30

↑在左侧的页面有这一问题

截屏2020-12-18 22 12 59

↑在右侧的页面也有这一问题

这个已经修复

tiensonqin commented 3 years ago

@tiensonqin

第四个问题,侧边栏打开新的页面,侧边栏不自动滚动到新打开的页面位置。当然,这也不能算是问题,但以前是滚动到新页面的位置,因为按照习惯是即将在新打开的页面操作的。

这个也已经修复。

xyhp915 commented 3 years ago

@tiensonqin ten 第二个问题,就是在滚动侧边栏页面时,如果滚动到顶或底时,会接着滚动左侧页面。我觉得这个交互是不必要的,以前也不存在这个情况,当然您可能有自己的考虑。

这个可能是最近的修改改变了行为,@xyhp915 你觉得这个交互要不要去掉?

这个是个权衡问题,咱们的sidebar宽度占比40%,如果冒然把这块滚动事件禁掉向上冒泡,可能带来的问题更多。

Pathsis commented 3 years ago

@tiensonqin ten 第二个问题,就是在滚动侧边栏页面时,如果滚动到顶或底时,会接着滚动左侧页面。我觉得这个交互是不必要的,以前也不存在这个情况,当然您可能有自己的考虑。

这个可能是最近的修改改变了行为,@xyhp915 你觉得这个交互要不要去掉?

这个是个权衡问题,咱们的sidebar宽度占比40%,如果冒然把这块滚动事件禁掉向上冒泡,可能带来的问题更多。

不知道这会带来什么问题呢?就我的个人体验来看,这个交互让我在滚动时小心翼翼,总是有种不顺畅的感觉啊,总是一不小心就把左侧的笔记滚动了位置。roam research也是巨大的siderbar条幅,但也没有这个交互。

xyhp915 commented 3 years ago

这个是个权衡问题,咱们的sidebar宽度占比40%,如果冒然把这块滚动事件禁掉向上冒泡,可能带来的问题更多。

不知道这会带来什么问题呢?就我的个人体验来看,这个交互让我在滚动时小心翼翼,总是有种不顺畅的感觉啊,总是一不小心就把左侧的笔记滚动了位置。roam research也是巨大的siderbar条幅,但也没有这个交互。

嗯,有一部分是技术上的问题,其实目前的交互行为并不是我们干涉的,是浏览器自身的行为,有部分原因是在提示用户,当前交互区域到达了边界,不能往下滑了(就像ios弹性滚动告知用户到达了边界),本身应该是没什么问题,包括自己使用过程中也没觉得太突兀,但还是想了下你为什么会有”小心翼翼“的感觉,你是不是大量在侧边栏长篇幅编辑内容,频繁在主区域与侧边栏切换(对你来讲,这两个区域几乎就是平等的地位),而不像我大部分时候只是把这个侧边栏当作目录来使用? 如果是这样,那我们是需要再想想了!

tiensonqin commented 3 years ago

@xyhp915 这个的确是个问题,因为最近打开的页面是在侧边栏的顶部,如果当前区域是在右侧栏的话,滑动到顶部这时肯定是不想滑动左侧栏的,因为很有可能这时候需要参考左侧栏区域的内容。

Pathsis commented 3 years ago

这个是个权衡问题,咱们的sidebar宽度占比40%,如果冒然把这块滚动事件禁掉向上冒泡,可能带来的问题更多。

不知道这会带来什么问题呢?就我的个人体验来看,这个交互让我在滚动时小心翼翼,总是有种不顺畅的感觉啊,总是一不小心就把左侧的笔记滚动了位置。roam research也是巨大的siderbar条幅,但也没有这个交互。

嗯,有一部分是技术上的问题,其实目前的交互行为并不是我们干涉的,是浏览器自身的行为,有部分原因是在提示用户,当前交互区域到达了边界,不能往下滑了(就像ios弹性滚动告知用户到达了边界),本身应该是没什么问题,包括自己使用过程中也没觉得太突兀,但还是想了下你为什么会有”小心翼翼“的感觉,你是不是大量在侧边栏长篇幅编辑内容,频繁在主区域与侧边栏切换(对你来讲,这两个区域几乎就是平等的地位),而不像我大部分时候只是把这个侧边栏当作目录来使用? 如果是这样,那我们是需要再想想了!

是啊,当我在左侧打开有待深度整理的长幅笔记,在右边栏打开条目撰写第二阶段的笔记时,就会是这样的。这个使用场景应该是roam笔记的典型场景啊。

当前交互区域到达了边界,不能往下滑了(就像ios弹性滚动告知用户到达了边界),本身应该是没什么问题

但是,当侧边栏到达底部,如果像ios的弹跳提示,不是弹跳下侧边栏吗?不应该接着移动左侧笔记的页面啊。这种事件告知必须是“无效果”的、内部的告知,而不是继续滚动其他页面这样的”有效果的”、外部的告知啊。就像我在电脑中同时打开两个word文档,一个是原文,一个是译文,当我滚动译文页面到达边界,事件告知仅仅应该发生在译文这个文档内,而不是在译文页面滚动到尽头后接着滚动原文的页面啊。

Pathsis commented 3 years ago

@xyhp915 这个的确是个问题,因为最近打开的页面是在侧边栏的顶部,如果当前区域是在右侧栏的话,滑动到顶部这时肯定是不想滑动左侧栏的,因为很有可能这时候需要参考左侧栏区域的内容。

是啊,在左侧打开待整理的笔记,然后shift-click以在侧边看打开许多条目,并在侧边栏撰写第二阶段的深度笔记,这应该是roam类笔记的典型场景啊。侧边栏绝不仅是目录区域啊。

xyhp915 commented 3 years ago

@xyhp915 这个的确是个问题,因为最近打开的页面是在侧边栏的顶部,如果当前区域是在右侧栏的话,滑动到顶部这时肯定是不想滑动左侧栏的,因为很有可能这时候需要参考左侧栏区域的内容。

是啊,在左侧打开待整理的笔记,然后shift-click以在侧边看打开许多条目,并在侧边栏撰写第二阶段的深度笔记,这应该是roam类笔记的典型场景啊。侧边栏绝不仅是目录区域啊。

嗯,这其实算是多平级窗口场景,其中一个窗口的滚动事件不该影响到另一个窗口的!!我们会考虑个折中的技术方案来尽快处理下这个问题,感谢 🙏

Pathsis commented 3 years ago

@xyhp915 这个的确是个问题,因为最近打开的页面是在侧边栏的顶部,如果当前区域是在右侧栏的话,滑动到顶部这时肯定是不想滑动左侧栏的,因为很有可能这时候需要参考左侧栏区域的内容。

是啊,在左侧打开待整理的笔记,然后shift-click以在侧边看打开许多条目,并在侧边栏撰写第二阶段的深度笔记,这应该是roam类笔记的典型场景啊。侧边栏绝不仅是目录区域啊。

嗯,这其实算是多平级窗口场景,其中一个窗口的滚动事件不该影响到另一个窗口的!!我们会考虑个折中的技术方案来尽快处理下这个问题,感谢 🙏

@tiensonqin ten @xyhp915 页面滚动的这个问题,不知道考虑得如何啦?如果必须保留的话,能否提供一个选项呢?

tiensonqin commented 3 years ago

@mkyos 这个等桌面端这两天出来以后会解决,暂时有个 PR: https://github.com/logseq/logseq/pull/971

tiensonqin commented 3 years ago

单独这个问题已修复

Pathsis commented 3 years ago

单独这个问题已修复

这个问题是哪个问题呢?是页面滚动逻辑这个问题吗?我刚测试了下,008没有改变啊

tiensonqin commented 3 years ago

页面滚动那个 PR 还没合并, 我是说主题

On Wed, Feb 17, 2021 at 2:19 PM mkyos notifications@github.com wrote:

单独这个问题已修复

这个问题是哪个问题呢?是页面滚动逻辑这个问题吗?我刚测试了下,008没有改变啊

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/logseq/logseq/issues/952#issuecomment-780331467, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADU7QI4MLEJVLJ7LZSW5XTS7NNYLANCNFSM4VA3GD4Q .

Pathsis commented 3 years ago

页面滚动那个 PR 还没合并, 我是说主题 On Wed, Feb 17, 2021 at 2:19 PM mkyos @.***> wrote: 单独这个问题已修复 这个问题是哪个问题呢?是页面滚动逻辑这个问题吗?我刚测试了下,008没有改变啊 — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#952 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADU7QI4MLEJVLJ7LZSW5XTS7NNYLANCNFSM4VA3GD4Q .

好滴!