Closed Eve-Sama closed 2 years ago
antvis/G2#3722 antvis/G2Plot#2969
@lxfu1 看下 customContent 之前为什么这么写
这个PR的实现还是有点问题, 我再看下.
算了, 我不想研究了, 这后面涉及到你们G2的一整套生态体系. 包括@antv/dom-util
. 反正问题我已经说清楚了, 最核心的代码块也告诉你们了, 只是这样改会引起很多其他问题, 包括边界检测失效等, 这一个bug一个bug的改, 成本太大了, 我不想研究了.
Checklist
npm test
passesDescription of change
tooltip
为customContent
时, transition未生效. 详情可以看下面这张gif https://file.qingflow.com/uploads/file/ce8f0f29-15f0-4439-a980-4b58ec20f3b8.gif改动稍微有一点大, 已经通过所有单测. 官方可以再详细测下, 我会在代码内容区写上相关评论以帮助你们更好理解此次PR的逻辑.
在修改之后, 在有
customContent
的场景下, DOM会是如下结构.g2-tooltip-position
仅仅用来作定位使用.另外, 因为component没有独立的网站, 我是通过jest-electron进行测试的, 这种改法是否真的会生效我并不确定, 我只是按照理论进行修改. 需要官方团队再次验证.