uiwjs / react-amap

基于 React 封装的高德地图组件,帮助你轻松的接入地图到 React 项目中。
https://uiwjs.github.io/react-amap
MIT License
428 stars 70 forks source link

为什么labelMarker的event是小写的,而其他的是大写的 #337

Closed starInEcust closed 12 months ago

starInEcust commented 1 year ago

为啥label是 onMousemove,而massMark是onMouseMove。我把labe的改成onMouseOver貌似也没啥问题

useEventProperties<AMap.LabelMarker, UseLabelMarker>(labelMarker!, props, [
    'onMousedown',
    'onMouseup',
    'onTouchstart',
    'onTouchmove',
    'onTouchend',
    'onClick',
    'onMousemove',
    'onMouseover',
    'onMouseout',
  ]);
jaywcjlove commented 1 year ago

@starInEcust 很尴尬,这个是个问题, 官方事件名称为 mousemove,为了保持跟 react 官方事件一致,做了 props 处理

发现 LabelMarker 的文档,和 ts 注释是正确的, 但是处理的时候是错误的。

现在做了修改,类型上支持了 老的事件 props(如:onMousedown),同时也支持了修正了事件名称的命名 onMouseDown

https://github.com/uiwjs/react-amap/blob/f887b869c071e99622a82f3dc89190c0fbbb5fda/packages/label-marker/src/index.tsx#L10-L49

starInEcust commented 1 year ago

@jaywcjlove 感恩,回复太快了,生产队的驴都没你这么卷- -

jaywcjlove commented 1 year ago

@starInEcust 能力有限,主打就是服务态度好 :)