Skip to content

Conversation

@vaebe
Copy link
Collaborator

@vaebe vaebe commented Oct 28, 2022

  • 修复 range-date-picker-pro v-model 测试用例错误,设置日期时大于20设置为当前日期,所以utils相关函数不应该大于20时-1

截屏2022-10-28 下午10 21 14

这个 getDateIndex 函数为什么要这么计算呢?哪位大佬明白的告知下

@github-actions
Copy link

PR preview has been successfully built and deployed to https://vue-devui-pr-1424.surge.sh.

@kagol
Copy link
Member

kagol commented Nov 2, 2022

  • 修复 range-date-picker-pro v-model 测试用例错误,设置日期时大于20设置为当前日期,所以utils相关函数不应该大于20时-1
截屏2022-10-28 下午10 21 14

这个 getDateIndex 函数为什么要这么计算呢?哪位大佬明白的告知下

看不太懂,看名字这个函数的作用是获取某个日期在当前月份表格中的索引位置?

@kagol
Copy link
Member

kagol commented Nov 2, 2022

  • 修复 range-date-picker-pro v-model 测试用例错误,设置日期时大于20设置为当前日期,所以utils相关函数不应该大于20时-1
截屏2022-10-28 下午10 21 14

这个 getDateIndex 函数为什么要这么计算呢?哪位大佬明白的告知下

@vaebe 我看懂了,这个函数是获取某个日期在月份表格中的索引,如下图:

image

获取的方式两部分相加

  • 目标日期所在月份第一周中在上一个月那一部分的索引
  • 目标日期在当前月份索引

如下图:

image

比如:getDateIndex(new Date('2022-11-11')),得到的是12

image

dayjs(date).subtract(date.getDate() - 1, 'day').day() + date.getDate() - 1
<=>
- 第一周中在上一个月的索引:dayjs(date).subtract(date.getDate() - 1, 'day').day() +
- 目标日期在当前月份索引 date.getDate() - 1

@kagol kagol merged commit 68cce74 into DevCloudFE:dev Nov 2, 2022
@kagol kagol added the unit-test Unit test label Nov 2, 2022
@vaebe vaebe deleted the zcj-fix-test branch November 3, 2022 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants