xrutayisire / react-js-cron

A React cron editor built with antd
https://xrutayisire.github.io/react-js-cron/?path=/docs/reactjs-cron--demo
MIT License
247 stars 106 forks source link

Pressing clear inside 'Select' does not show desired result. #18

Closed ashishRudder closed 1 year ago

ashishRudder commented 2 years ago

Describe the bug When clicked on clear button in minutes or hours Select, the value inside don't get cleared.

To Reproduce Steps to reproduce the behavior:

  1. Go to 'Cron widget'
  2. Click on 'minutes dropdown'
  3. Add at least 5 items.
  4. Close the dropdown
  5. Click on clear/X button on the right.
  6. See bug where the values don't get clear.

Expected behavior Once clicked on clear, the values should get removed.

CodeSandbox https://codesandbox.io/s/cron-antd-rcjv9

Versions (please complete the following information):

Additional context The issue is not present when switching to antd version 4.17.2

Demo: https://user-images.githubusercontent.com/74043614/153365290-5a82cb1a-d727-476c-8a9d-ded9a547e238.mov

xrutayisire commented 2 years ago

Hello,

Thanks for the issue. It clearly seems like v4.18.0 of antd introduce the bug in the Cron component. I will take a look and try to fix this ASAP.

Xavier

xrutayisire commented 2 years ago

There is an open issue on antd side. I will wait to see if they can revert this new behavior (bug for me) or not.

cf: https://github.com/ant-design/ant-design/issues/33790

xrutayisire commented 1 year ago

It's closed on antd side, so now you should not have the bug anymore: https://github.com/ant-design/ant-design/issues/33790