search

uiwjs / react-heat-map

A lightweight calendar heatmap react component built on SVG, customizable version of GitHub's contribution graph.
https://uiwjs.github.io/react-heat-map
MIT License
209 stars 25 forks source link

RangeError: Invalid array length #24

Closed excxapp closed 3 years ago

excxapp commented 3 years ago

<g transform={translate(${leftPad}, ${topPad})}>

错误提示这一行，请问是什么问题

jaywcjlove commented 3 years ago

@excxapp 有实例？

excxapp commented 3 years ago

image 我的场景是，列表也，会多次重新渲染，popover上的。 方便给个微信啥的不

jaywcjlove commented 3 years ago

@excxapp 提供一个实例我看看。

excxapp commented 3 years ago

我项目里面用的用的，没有拆解的出来。 具体就是，我reacthook里面 会重新渲染 value

excxapp commented 3 years ago

image image 这是用到的两部分

jaywcjlove commented 3 years ago

@excxapp 你抽出来放到一个 repo 中，我方便测试，截图我怎么调试呢？

jaywcjlove commented 3 years ago

@excxapp 你使用的 Tooltip 有没有破坏 svg ？

excxapp commented 3 years ago

tooltip是ant的，不是你们那个，因为你们那个层级太低。我注视掉tooltips一样抱错。

jaywcjlove commented 3 years ago

@excxapp 我不太知道你什么原因，没有示例，没有办法解决

excxapp commented 3 years ago

你一个页面多次渲染一个通过按钮点击改变value，value每次都一样的情况，你测试下应该就能重现了

jaywcjlove commented 3 years ago

@excxapp 我看了官网示例，没有问题，如果你没有提供示例，我没有办法解决你的问题。

jaywcjlove commented 3 years ago

@excxapp 你可以通过 npx create-kkt my-app 创建一个简单的工程，重现你的bug，然后我帮你调试一下

excxapp commented 3 years ago

我自己修改了源码加了判断是可以了

jaywcjlove commented 3 years ago

@excxapp 好吧，欢迎提交 PR

excxapp commented 3 years ago

@excxapp 好吧，欢迎提交 PR

好的，我系统babel没升级，所以我没跑你这个项目。我改了2个地方 LablesMonth.tsx line-22 if (monthLables === false) return []; 改为 if (monthLables === false || colNum<1) return []; SVG.tsx line 104

{gridNum&&gridNum >0&&[...Array(gridNum)].map((_, idx) => {

jaywcjlove commented 3 years ago

@excxapp Upgrade @uiw/react-heat-map@1.3.4