notification icon indicating copy to clipboard operation
notification copied to clipboard

feat(show-progress): add show progress support

Open yociduo opened this issue 10 months ago • 1 comments

close https://github.com/ant-design/ant-design/issues/48125

yociduo avatar Apr 09 '24 07:04 yociduo

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
notification ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 11, 2024 3:36am

vercel[bot] avatar Apr 09 '24 07:04 vercel[bot]

  1. hover 到 notificiation 上时,progress 应该是暂停,总之不应该消失。看上去还需要一个 pauseOnHover: boolean;
  2. 是否应该是反向 progresss?

参考 https://fkhadra.github.io/react-toastify/introduction/

afc163 avatar May 07 '24 10:05 afc163

  1. hover 到 notificiation 上时,progress 应该是暂停,总之不应该消失。看上去还需要一个 pauseOnHover: boolean;
  2. 是否应该是反向 progresss?

参考 https://fkhadra.github.io/react-toastify/introduction/

  1. 是沿用旧逻辑,我先让它不消失吧。这个新的属性是新开个pr改,还是就在当前这个改掉。我觉得加了这个的话,closeOnClick: boolean也应该加上。
  2. 这个倒是正反的都看到过,按你说的我改一下,改成反向的。
  3. showProgress -> showProgressBar 这个属性这样改是不是好一点

yociduo avatar May 08 '24 01:05 yociduo

showProgress 就行了。

afc163 avatar May 08 '24 02:05 afc163

hover 后现在是暂停的,但是鼠标离开后重新计时了,预期应该是从上次时刻继续。

afc163 avatar May 11 '24 01:05 afc163

hover 后现在是暂停的,但是鼠标离开后重新计时了,预期应该是从上次时刻继续。

这个更新了,顺便加了一下测试用例

yociduo avatar May 11 '24 03:05 yociduo