claneo

Results 9 comments of claneo

This is my `babel-plugin-import` config: ```js [ 'import', { libraryName: '@jbuschke/formik-antd', style: libraryName => { let name = libraryName.replace('@jbuschke/formik-antd/es/', ''); if (name === 'FormikDebug') return false; if (name === 'SubmitButton')...

@jannikbuschke It's correct. Half of the bundle size comes from `antd`'s icon library, which really annoyed users over the past year. BTW, it seems like that setting `sideEffects` field of...

类型定义的改动感觉可以权衡下?据我所知很多社区的库也是不承诺类型定义变动时保持兼容的(甚至 TS 自身用的都不是 semver)

@PengJiyuan 在源码里搜了一下,findDOMNode 出现的次数也不算太多,看起来都能处理一下 1. 有个叫 getNodeHeight 的函数,看了下传参已经是一个 HTMLElement 了,没必要用 findDOMNode 了 2. ResizeObserverComponent 是个内部的组件,应该可以直接换成读 children 的 ref 3. 有些组件里用到了 findDOMNode(getPopupContainer()),这里可以判断下,如果 getPopupContainer 返回的是个 ReactElement 再使用 findDOMNode,并在文档里提示 4. 最后一个就是 Trigger 了,首先里面有一半是 findDOMNode(this.triggerRef),可以直接去掉。然后麻烦的就是获取...

我能想到的一个坑是在 Modal/Drawer 里面的情况,里面的 Spin 就不需要读外面 Spin 的 loading 了。 以及其他业务代码里可能有的 absolute、relative、transform 等元素位置有偏移的情况

@yinkaihui 再添加一个为什么建议直接用 useSyncExternalStore 的理由 看这个 demo:https://codesandbox.io/s/elastic-satoshi-73nwkm?file=/demo.js 首次渲染,控制台打印的日志正常: ``` field: data[0], value: 1 field: data[1], value: 2 field: data[2], value: 3 ``` 然后把第一行的表单项删掉,看控制台就会发现发生了两次渲染,而第一次的结果是不对的 ``` field: data[0], value: 3 field: data[1], value:...

> How was this module imported into your code? This a file in `monaco-editor`, my code is just: ```js import * as monaco from 'monaco-editor';` ``` > Do you have...

我发现表头吸顶其实并不需要依赖 `react-sticky`,直接给表头加个 `positin: sticky` 就行,但需要先把 `.arco-table-content-scroll` 和 `.arco-table-content-inner` 两个元素的 `overflow` 样式去掉,目前还不知道会有什么影响

getBoundingClientRect 确实在存在动画的情况下不准,先换成 +1 了