Skip to content

Conversation

zombieJ
Copy link
Member

@zombieJ zombieJ commented Jun 20, 2020

  1. 选结束时间直接关闭浮层,完成选择。
  2. 选开始时间会保持浮层,需要再点结束时间,这样和初始选择的习惯保持一致。

发现还有一些边界情况,调整为:

  • 开始时间关闭,如果结束时间可以输入则打开结束时间。
  • 结束时间关闭,如果开始时间为空则打开开始时间。
    • 结束时间打开的开始时间,关闭后不打开结束时间
    • 结束时间打开的开始时间,如果开始时间晚于结束时间,则重置结束时间,同时再次打开结束时间

ref ant-design/ant-design#25120

@vercel
Copy link

vercel bot commented Jun 20, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/react-component/picker/5b59gb3u6
✅ Preview: https://picker-git-logic.react-component.vercel.app

@codecov
Copy link

codecov bot commented Jun 20, 2020

Codecov Report

Merging #89 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #89   +/-   ##
=======================================
  Coverage   99.55%   99.55%           
=======================================
  Files          42       42           
  Lines        1792     1796    +4     
  Branches      538      534    -4     
=======================================
+ Hits         1784     1788    +4     
  Misses          6        6           
  Partials        2        2           
Impacted Files Coverage Δ
src/hooks/useTextValueMapping.ts 100.00% <ø> (ø)
src/RangePicker.tsx 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b921c3...66afd3c. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant