Eden Wang

Results 9 comments of Eden Wang

The root cause is that the component can't get the event when the child node width and height change. For example, the child component is an image, After image loading,...

> 默认已经是有这个能力了,要加这个 API 是为了停用这个功能么? 现在的实现还是有点不一样的,现在不是暂停计时器,而是重新重新计时,加了进度条这个PR后会发现这个行为有些奇怪

> > 现在的实现还是有点不一样的,现在不是暂停计时器,而是重新重新计时,加了进度条这个PR后会发现这个行为有些奇怪 > > 如果是这样的话,看起来应该是没有进度条时 hover 重置,有进度条时 hover 暂停。作为默认行为就好了?感觉它不应该是一个透出的 API 到用户侧。 我觉得不管有没有进度条,行为应该是统一的,当时加 progress bar 的时候就觉得 hover 的时候重置会有一些奇怪。如果需要改成默认行为的话,还需要改一下基础库,我稍后更新。

> the preview here https://preview-48353-ant-design.surge.sh/components/notification#notification-demo-show-progress doesn't show any progress indicator Yes, it's work in progress. I'm waiting for the dependence merged. The demo for rc-notification https://notification-git-fork-yociduo-feat-show-progess-react-component.vercel.app/demo/hooks

> ![image](https://private-user-images.githubusercontent.com/27722486/321151847-ead75c0a-c84c-459f-b101-114e2649e96c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTMxNzEyODEsIm5iZiI6MTcxMzE3MDk4MSwicGF0aCI6Ii8yNzcyMjQ4Ni8zMjExNTE4NDctZWFkNzVjMGEtYzg0Yy00NTlmLWIxMDEtMTE0ZTI2NDllOTZjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA0MTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNDE1VDA4NDk0MVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWQxODQyMjg4ZGUzNmRhOTllNTNkZGNjYzI1MDI4OTM3YjA1NDM3MmQ3NWRjNWUzMzg2MjM2Y2JjYmViYzJjM2QmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.AjJKNACQOrBp-zYueDFBf0VzFV0NawwP9a4sC2pp1C8) > > 可以参考这个设计 ![image](https://github.com/ant-design/ant-design/assets/8317101/fc779bf9-8f6b-40b1-b536-a8c8ef26fe02) 更新了,不过需要等rc-notification库更新

> @yociduo lint 报错了,需要处理一下 修复了,不过需要先处理组件那边的更新。

https://github.com/ant-design/ant-design/issues/46200 https://github.com/react-component/input/pull/54 It is related to this change, react-number-format can't get the real target.

> 1. hover 到 notificiation 上时,progress 应该是暂停,总之不应该消失。看上去还需要一个 `pauseOnHover: boolean;`。 > 2. 是否应该是反向 progresss? > > 参考 https://fkhadra.github.io/react-toastify/introduction/ 1. 是沿用旧逻辑,我先让它不消失吧。这个新的属性是新开个pr改,还是就在当前这个改掉。我觉得加了这个的话,`closeOnClick: boolean`也应该加上。 2. 这个倒是正反的都看到过,按你说的我改一下,改成反向的。 3. `showProgress` -> `showProgressBar` 这个属性这样改是不是好一点

> hover 后现在是暂停的,但是鼠标离开后重新计时了,预期应该是从上次时刻继续。 这个更新了,顺便加了一下测试用例