lowcode-engine icon indicating copy to clipboard operation
lowcode-engine copied to clipboard

feat(plugin-outline-pane): add MouseEvent params in onMoveHook

Open Ronchan0805 opened this issue 2 years ago • 5 comments

新增在onMoveHook钩子中返回鼠标事件参数MouseEvent

Ronchan0805 avatar Feb 04 '23 07:02 Ronchan0805

Codecov Report

Base: 86.11% // Head: 86.11% // No change to project coverage :thumbsup:

Coverage data is based on head (1d2dae5) compared to base (f4868b8). Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1530   +/-   ##
=======================================
  Coverage   86.11%   86.11%           
=======================================
  Files         197      197           
  Lines        8579     8579           
  Branches     2062     2062           
=======================================
  Hits         7388     7388           
  Misses       1107     1107           
  Partials       84       84           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Feb 06 '23 01:02 codecov[bot]

麻烦描述下需要这个的具体场景

JackLian avatar Feb 06 '23 01:02 JackLian

实现自由画布(自由拖拽和缩放)一个想法是在组件层面去做,这样做的想法在于尽可能不去改动引擎。研究发现物料协议中在advanced里提供了包括onResize,onMoveHook在内的诸多接口,可以利用回调里的参数以及官方已经提供的Node节点相关方法实现组件的自由布局。目前缩放在已有引擎能力上已经可以实现,但由于拖拽时onMoveHook缺少鼠标MouseEvent相关参数,自由拖拽的定位难以确定。

Ronchan0805 avatar Feb 06 '23 03:02 Ronchan0805

那onHoverHook是不是也需要加一下?也看看触发时机是不是你想要的,有可能现在onMoveHook这个的时机不太符合你诉求也说不定

JackLian avatar Feb 06 '23 03:02 JackLian

1.分支指向改一下,指向 develop

2.commit 保留一个

3.这里的文档/协议也需要改一下,https://lowcode-engine.cn/site/docs/specs/material-spec#22-%E4%BD%8E%E4%BB%A3%E7%A0%81%E8%A7%84%E8%8C%83 在这个文件里面:https://github.com/alibaba/lowcode-engine/blob/main/docs/docs/specs/material-spec.md

liujuping avatar Feb 06 '23 03:02 liujuping

由于 PR 内容暂时还没有更新,该 PR 先关闭,有需要再重新提 PR。

liujuping avatar Mar 10 '23 08:03 liujuping