Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: dateSelect month #907

Merged
merged 6 commits into from
Jun 24, 2022
Merged

fix: dateSelect month #907

merged 6 commits into from
Jun 24, 2022

Conversation

edison-hm
Copy link
Contributor

修复DatePIckerView,DatePicker,DateSelect 组件 mode=month 时的 bug。

@vercel
Copy link

vercel bot commented Jun 16, 2022

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

Name Status Preview Updated
zarm ✅ Ready (Inspect) Visit Preview Jun 23, 2022 at 7:40AM (UTC)

@codecov
Copy link

codecov bot commented Jun 16, 2022

Codecov Report

Merging #907 (3c24dac) into master (9c18470) will increase coverage by 0.86%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #907      +/-   ##
==========================================
+ Coverage   82.99%   83.86%   +0.86%     
==========================================
  Files         183      183              
  Lines        5111     5143      +32     
  Branches     1129     1144      +15     
==========================================
+ Hits         4242     4313      +71     
+ Misses        861      823      -38     
+ Partials        8        7       -1     
Impacted Files Coverage Δ
...ges/zarm/src/date-picker-view/utils/parseState.tsx 100.00% <100.00%> (+6.25%) ⬆️
packages/zarm/src/affix/Affix.tsx 57.94% <0.00%> (+7.94%) ⬆️
...kages/zarm/src/date-picker-view/DatePickerView.tsx 64.52% <0.00%> (+18.37%) ⬆️

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 9c18470...3c24dac. Read the comment docs.

@vercel vercel bot temporarily deployed to Preview – zarm June 16, 2022 08:32 Inactive
@vercel vercel bot temporarily deployed to Preview – zarm June 16, 2022 14:40 Inactive
@JeromeLin
Copy link
Collaborator

以下场景有问题

<DateSelect
        className="test-dateSelect"
        title="选择日期"
        placeholder="请选择日期"
        mode="time"
        min="2020-05-16 12:33"
        max="2027-05-15 05:20"
        value={value}
        onOk={(value) => {
          console.log('DateSelect onOk: ', value);
          setValue(value);
        }}
      />

image

@vercel vercel bot temporarily deployed to Preview – zarm June 19, 2022 16:11 Inactive
@JeromeLin
Copy link
Collaborator

wecom-temp-8e126e67b82f3f10119e2cead319d7c6

@vercel vercel bot temporarily deployed to Preview – zarm June 23, 2022 07:40 Inactive
@JeromeLin JeromeLin merged commit 810162d into master Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants