taroify icon indicating copy to clipboard operation
taroify copied to clipboard

🎸 [Feature]: FloatingPanel 提供拖拽相关事件

Open felix9ia opened this issue 9 months ago • 6 comments

这个功能解决了什么问题?

可以让FloatingPanel 的拖拽动作被外界感知,然后做出相应的动作

你期望的组件设计是怎样的?

binddragstart 滑动开始事件 binddragging 滑动事件 binddragend 滑动结束事件

vant-ui 提供的 height-change 事件,无法捕捉时机前后 https://vant-ui.github.io/vant/#/zh-CN/floating-panel

你想为这个功能创建一个 PR 吗?

None

felix9ia avatar May 09 '25 03:05 felix9ia

可以传入handleChange

Image

ta会在拖拽结束且高度发生变化后触发,并返回当前高度

Image

HyaCiovo avatar May 12 '25 06:05 HyaCiovo

感觉组件的文档比代码的api落后很多

HyaCiovo avatar May 12 '25 06:05 HyaCiovo

感觉组件的文档比代码的api落后很多

确实,文档里没有提及,我以为没有相关的事件,只是 handleChange 这个命名是不是有些不合适,onHeightChange好一些?

felix9ia avatar May 12 '25 06:05 felix9ia

感觉组件的文档比代码的api落后很多

确实,文档里没有提及,我以为没有相关的事件,只是 handleChange 这个命名是不是有些不合适,onHeightChange好一些?

确实onHeightChange合适一些。另外之前通过ref暴露出来的手动修改高度的方法好像没法触发这个事件,到底应不应该触发这个事件也要好好考虑一下。如果要对其vant-ui的话,这个讲起来不算是拖动改变的高度

Image

HyaCiovo avatar May 12 '25 06:05 HyaCiovo

感觉组件的文档比代码的api落后很多

确实,文档里没有提及,我以为没有相关的事件,只是 handleChange 这个命名是不是有些不合适,onHeightChange好一些?

确实onHeightChange合适一些。另外之前通过ref暴露出来的手动修改高度的方法好像没法触发这个事件,到底应不应该触发这个事件也要好好考虑一下。如果要对其vant-ui的话,这个讲起来不算是拖动改变的高度

Image

确实得考虑 从我使用组件的角度来看,我肯定希望高度只要发生了变化,事件就应该感知 其实 ref 这种调用和原本的 height 属性多少让我看起来有些困惑,我会下意识认为有了 height 了,为什么还需要 ref ?

felix9ia avatar May 12 '25 06:05 felix9ia

确实得考虑 从我使用组件的角度来看,我肯定希望高度只要发生了变化,事件就应该感知 其实 ref 这种调用和原本的 height 属性多少让我看起来有些困惑,我会下意识认为有了 height 了,为什么还需要 ref ?

我之前是认为应该用defaultHeight来代替height,手动修改height通过ref来进行,这样组件改动不大。如果是要通过直接修改props.height的方式来改变高度,那就还要通过react.state onChange来设计成一个受控组件了。

HyaCiovo avatar May 12 '25 07:05 HyaCiovo