dwqs / v2-datepicker

A simple datepicker component based Vue 2.x: https://dwqs.github.io/v2-datepicker/
MIT License
89 stars 21 forks source link

范围日期的一个建议和一个疑似 Bug。 #12

Closed Xeath closed 6 years ago

Xeath commented 6 years ago

建议

在开启左右联动的情况下,建议增加一个选项可以设置左边面板起始月份为上个月,比如说数据查询类就并不需要未来的日期,所以一般会选择禁止未来日期。也有一些业务也是需要未来日期,比如说行程类。

疑似 Bug

default 在开启左右联动的情况下,如果给范围日期定义一个默认值为相同月份,会导致左右面板的月份相同。 问题出在

if (!this.unlinkPanels && diff > MONTH) {
    rightDate = (0, _utils.nextMonth)(leftDate, 1);
}

其中的 diff > MONTH 我并不明白具体用意,这一段对于左右面板联动来说,好像是多余的,在开启左右面板联动的情况下,右面板应该是起始日期的下一个月

dwqs commented 6 years ago

@xiaopiao5789

建议增加一个选项可以设置左边面板起始月份为上个月

这个可以设置 defaultValue 属性 这个属性从 2.2.0 之后开始支持。相关初始化代码,如果你想把如果初始为3月与4月,可以这么做:

// 把 default-value 设置成上个月份的某个日期
<v2-datepicker-range :default-value="'2018-03-12'"></v2-datepicker-range>

你的这个 bug 是什么版本 如果不是最新版本 建议先升级下版本看看 这个问题我记得在某个版本已经修复过了

if (!this.unlinkPanels && diff > MONTH) {
    rightDate = (0, _utils.nextMonth)(leftDate, 1);
}

这部分是当传入默认值时 如果没有取消联动且两个默认日期的时间差超过一个月了 就会把右面板的初始值纠正为第一个初始值的下一月份

dwqs commented 6 years ago

@xiaopiao5789 我刚试了最新版本 截图确认是一个 bug 明天会发一个小版本修复

Xeath commented 6 years ago

@dwqs 感谢作者提醒,昨天并没有仔细查阅文档。

dwqs commented 6 years ago

@xiaopiao5789 截图的 bug 升级到 3.0.3试试

Xeath commented 6 years ago

@dwqs 这个只是修复了相同日期,但是我文中表示的是「相同月份」,不好意思,都怪我放错截图 default

dwqs commented 6 years ago

@xiaopiao5789 新开了个 issue: https://github.com/dwqs/v2-datepicker/issues/13 我看下这个问题