Skip to content
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

feat(useDrag): useDrag adds custom picture feature #2182

Merged
merged 13 commits into from Jul 12, 2023

Conversation

kakachake
Copy link
Contributor

@kakachake kakachake commented May 3, 2023

🤔 这个变动的性质是?

  • 新特性提交
  • 日常 bug 修复
  • 站点、文档改进
  • 演示代码改进
  • TypeScript 定义更新
  • 包体积优化
  • 性能优化
  • 功能增强
  • 国际化改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他改动(是关于什么的改动?)

🔗 相关 Issue

💡 需求背景和解决方案

  1. 新增useDrag自定义拖拽图片的功能
  2. 在DragOptions中新增以下参数:

DragOptions

参数 说明 类型 默认值
dragImage 自定义图像 DragImageOptions -

DragImageOptions

参数 说明 类型 默认值
image 图片 string | Element -
offsetX 水平偏移 number 0
offsetY 垂直偏移 number 0

📝 更新日志

语言 更新描述
🇺🇸 英文
🇨🇳 中文 新增useDrag自定义拖拽图片的功能

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

@CLAassistant
Copy link

CLAassistant commented May 3, 2023

CLA assistant check
All committers have signed the CLA.

@kakachake kakachake changed the title useDrag adds custom picture feature feat: useDrag adds custom picture feature May 3, 2023
@kakachake kakachake changed the title feat: useDrag adds custom picture feature feat(useDrag): useDrag adds custom picture feature May 3, 2023
@liuyib
Copy link
Collaborator

liuyib commented May 4, 2023

useDrop 中有配合 useDrag 处理文件的 onFiles, onUri 事件,都可以处理图片相关的逻辑。我感觉这里不需要新增这个 api。可以说出你的具体业务需求吗?利用现有 api 也许能实现

@liuyib liuyib self-requested a review May 4, 2023 02:45
@liuyib liuyib self-assigned this May 4, 2023
@kakachake
Copy link
Contributor Author

您可能理解错了,这个是拖动过程中跟随鼠标指针的图片,对应如下的api
https://developer.mozilla.org/zh-CN/docs/Web/API/DataTransfer/setDragImage

@liuyib
Copy link
Collaborator

liuyib commented May 4, 2023

确实理解错了,这个 api 也有些用处(可以用来在用户拖动时,显示图片给予用户一些提示)。

不过目前第一次拖动时,图片没加载,显示的浏览器默认的(如果复现不了就清空浏览器缓存后再拖动),如下所示:

2023-05-04.15.30.02.mov

@kakachake
Copy link
Contributor Author

按照您说的修改了一下

@liuyib
Copy link
Collaborator

liuyib commented May 4, 2023

按照您说的修改了一下

好的,辛苦,晚上了我再细看下

Co-authored-by: 云泥 <1656081615@qq.com>
@liuyib
Copy link
Collaborator

liuyib commented Jun 7, 2023

@crazylxr 有时间再看下呢,文档更新了~

@crazylxr crazylxr merged commit 8d91cab into alibaba:master Jul 12, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants