molecule icon indicating copy to clipboard operation
molecule copied to clipboard

Fixed two bugs of split component

Open slobber opened this issue 1 year ago • 5 comments

Fixed main panel height limit bug

disable panel content selectable when dragging splitters

slobber avatar Jun 21 '24 01:06 slobber

请问为什么要去掉 Editor 的高度限制呢, 之前是限制了 600 的高度, 现在 300 确实有点小。

shiqiWang0 avatar Jun 24 '24 01:06 shiqiWang0

主编辑器本不应该有最大限制呀 image 现在是拖拽一下水平分割条,就直接变这样,然后就再也拖不大了

slobber avatar Jun 25 '24 07:06 slobber

目前的设计是 :当有 Panel 的展示, Editor 会有最大的拖拽高度限制的。建议限制改为 600 , 再大就可以将 Panel 关闭就好。

shiqiWang0 avatar Jun 25 '24 08:06 shiqiWang0

这么设计的理由是什么?如果要保证Panel的大小,应该为Panel设置minSize,Editor部分不应该做限制。在大显示器上不管300/600的限制都是比较影响操作的。

1.x demo也没有这方面限制。

参考vscode 的 SplitView,除了在侧边栏使用到的 UserProfileDataView,其他所有Panel都没有设置过 maximunSize 为特定值,如果需要设置也是会设置 minimumSize。

slobber avatar Jun 26 '24 04:06 slobber

问题在于:

  1. 就你提供的 PR 来看,如果单纯将 editor 的 maxSize 去掉,由于我们 Panel 是没有最小限制的, 当 editor 最大化后,是需要联动 menuBar 的状态,否则会引入其他 bug; 如图所示: demo

  2. 根据你提供的 minSize 设置思路, 则需要设置在 Panel 上,这样的话仍无法达到想要在拖拽的情况下将 editor 最大化,目前只提供了关闭 Panel 的操作;

  3. 如果想达到 Panel 达到某一个最小阙值,实现自动收起, 目前我们还未支持,后续可以考虑优化掉, 也同时期待你的 PR

shiqiWang0 avatar Jun 26 '24 07:06 shiqiWang0