fix: expand animation

[shaokeyibb]

fix #102

[Enter-tainer]

https://developer.mozilla.org/zh-CN/docs/Web/API/Element/clientHeight
https://developer.mozilla.org/zh-CN/docs/Web/API/HTMLElement/offsetHeight

这俩height好像确实还有点区别（是否包含border），不过这里为啥需要设置两次height呀

[shaokeyibb]

clientHeight 等于是你能看到的那部分元素的高度（未展开高度），scrollHeight 是包括看不到的部分元素的高度（展开以后的高度），这里设置两次是为了强制浏览器渲染动画（参考：强制回流）

[shaokeyibb]

可能不太直观？但一直是前端一贯的做法，如果想搞得优雅一点其实可以换成 requestAnimationFrame 函数调用

[Enter-tainer]

话说这里之前的写法是怎么理解呀，先设置成100%，再设置成scrollHeight

[shaokeyibb]

突然意识到其实上面调用 const offsetHeight = commentMain.offsetHeight; 的时候已经强制回流一次了，那这里其实确实可以不用写两遍

[shaokeyibb]

话说这里之前的写法是怎么理解呀，先设置成100%，再设置成scrollHeight

可以理解为先展开，然后再播放动画？

[Enter-tainer]

话说这里之前的写法是怎么理解呀，先设置成100%，再设置成scrollHeight

可以理解为先展开，然后再播放动画？

但是之前的版本在展开的时候，不是没有动画吗😂

[shaokeyibb]

话说这里之前的写法是怎么理解呀，先设置成100%，再设置成scrollHeight

可以理解为先展开，然后再播放动画？

但是之前的版本在展开的时候，不是没有动画吗😂

是...这里先前版本其实是不 work 的，但是代码一直留在这里，所以我才说是一个已知问题（

[Enter-tainer]

okk 所以目前看起来是把 100% 那句话去掉之后，动画就回来了？

[shaokeyibb]

okk 所以目前看起来是把 100% 那句话去掉之后，动画就回来了？

确实，我目前本地测试是 ok 的（然而我目前不是很清楚它怎么就 work 了）

[Enter-tainer]

LGTM