-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
停靠栏面板支持鼠标悬停展开 #3499
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
Comments
终于来了,等到白头了 |
期待已久的特性啊,1.2之前就希望有了。 |
@Vanessa219 ...data?.forEach... export const getAllDocks = () => {
const docks: IDockTab[] = [];
window.siyuan.config.uiLayout.left.data?.forEach((item: IDockTab[]) => {
item.forEach((dock: IDockTab) => {
docks.push(dock);
});
});
window.siyuan.config.uiLayout.right.data?.forEach((item: IDockTab[]) => {
item.forEach((dock: IDockTab) => {
docks.push(dock);
});
});
window.siyuan.config.uiLayout.top.data?.forEach((item: IDockTab[]) => {
item.forEach((dock: IDockTab) => {
docks.push(dock);
});
});
window.siyuan.config.uiLayout.bottom.data?.forEach((item: IDockTab[]) => {
item.forEach((dock: IDockTab) => {
docks.push(dock);
});
});
return docks;
}; |
@Soltus 稍等,还没写完,欢迎 close 后再来测试。谢谢 😊 |
@Vanessa219 还有一个小建议,我使用模态弹窗时,停靠栏仍响应鼠标悬停展开,虽然目前看不出来有什么负面影响 |
差不多吧,使用的是UI库的Modal组件。虽然目前看没有副作用,但由于遮罩是半透明的,停靠栏响应鼠标事件但实际上需要先关闭遮罩层才能操作,存在不一致性 |
@mozhux 目前浮动状态下鼠标移除面板,面板就会消失。点击那个箭头可能会有点压力,后面再看看怎么改进。 |
|
@Vanessa219 左右浮窗的高度不一致,是有什么考量吗 |
动画卡顿不流畅 |
开启悬停再隐藏边栏后,想点击文档右上角的 |
支持一个按钮放在停靠栏上方,还要想想停靠栏隐藏时按钮怎么放。 |
楼上有提到避免遮挡 |
使用默认主题么? |
左右可以做一致 右侧也遮挡 |
如果隐藏了停靠栏就缩小触发点? |
考虑到右侧面板内容比较充实且右侧被遮挡内容比较少。 |
嗯嗯隐藏后触发区域可以靠下一些,避开左上方的面包屑和右上方的 |
牺牲几十个像素我觉得可以接受的,左右面板对称,逻辑上就比较一致,或者左侧高度也不减,但是触发区域降到面包屑以下 |
194px,等后续用户反馈再看看,毕竟反链还蛮占用位置的。 |
|
“钉住”按钮的位置和“文档树”“大纲”等按钮平级了,有可能造成困惑,或许可以调整下 |
|
#7363 第二点 |
https://ld246.com/article/1637906203921
The text was updated successfully, but these errors were encountered: