Reset input value when Picker close (#28)

* reset when closed

* add test case

Author: zombieJ

src/Picker.tsx

@@ -326,6 +326,8 @@ function InnerPicker<DateType>(props: PickerProps<DateType>) {
 
       if (!valueTexts.length || valueTexts[0] === '') {
         triggerTextChange('');
+      } else if (!valueTexts.includes(text)) {
+        resetText();
       }
     }
   }, [mergedOpen, valueTexts]);

src/RangePicker.tsx

@@ -614,9 +614,13 @@ function InnerRangePicker<DateType>(props: RangePickerProps<DateType>) {
 
       if (!startValueTexts.length || startValueTexts[0] === '') {
         triggerStartTextChange('');
+      } else if (!startValueTexts.includes(startText)) {
+        resetStartText();
       }
       if (!endValueTexts.length || endValueTexts[0] === '') {
         triggerEndTextChange('');
+      } else if (!endValueTexts.includes(endText)) {
+        resetEndText();
       }
     }
   }, [mergedOpen, startValueTexts, endValueTexts]);

src/hooks/useValueTexts.ts

@@ -2,17 +2,15 @@ import * as React from 'react';
 import { GenerateConfig } from '../generate';
 import { Locale } from '../interface';
 
+interface ValueTextConfig<DateType> {
+  formatList: string[];
+  generateConfig: GenerateConfig<DateType>;
+  locale: Locale;
+}
+
 export default function useValueTexts<DateType>(
   value: DateType | null,
-  {
-    formatList,
-    generateConfig,
-    locale,
-  }: {
-    formatList: string[];
-    generateConfig: GenerateConfig<DateType>;
-    locale: Locale;
-  },
+  { formatList, generateConfig, locale }: ValueTextConfig<DateType>,
 ) {
   return React.useMemo<string[]>(() => {
     if (!value) {

tests/picker.spec.tsx

@@ -587,4 +587,21 @@ describe('Picker.Basic', () => {
 
     errSpy.mockRestore();
   });
+
+  it('close to reset', () => {
+    const wrapper = mount(
+      <MomentPicker defaultValue={getMoment('2000-01-01')} />,
+    );
+
+    wrapper.openPicker();
+    wrapper.find('input').simulate('change', {
+      target: {
+        value: 'aaaaa',
+      },
+    });
+    expect(wrapper.find('input').props().value).toEqual('aaaaa');
+
+    wrapper.closePicker();
+    expect(wrapper.find('input').props().value).toEqual('2000-01-01');
+  });
 });