search

loper7 / DateTimePicker

⭐🎉一个高颜值日期时间选择器；极简API，内置弹窗，支持农历日期显示，适配深色模式，可动态配置样式及主题，选择器支持完全自定义UI。
1.09k stars 121 forks source link

关于setMaxMillisecond的Bug #47

Closed seeeeal closed 2 years ago

seeeeal commented 2 years ago

设备：Pixel4 系统版本：Android12

场景：Demo中的CardDatePickerDialog Max Value: 2022-01-06 15:47:47 Min Value: 1990-09-10 15:48:01 Def Value: 2000-12-22 15:48:10

点击Show Dialog

默认时间将显示在2000年1月1日15时0分10秒

经过观察发现只要Def value的Month或者Day > Max Value的Month或者Day，显示便会出错

seeeeal commented 2 years ago

Screenshot_20220110-160407 Screenshot_20220110-160410

loper7 commented 2 years ago

设备：Pixel4 系统版本：Android12

场景：Demo中的CardDatePickerDialog Max Value: 2022-01-06 15:47:47 Min Value: 1990-09-10 15:48:01 Def Value: 2000-12-22 15:48:10

点击Show Dialog

默认时间将显示在2000年1月1日15时0分10秒

经过观察发现只要Def value的Month或者Day > Max Value的Month或者Day，显示便会出错

0.5.3 已处理

HeJingWei commented 2 years ago

还是有一点问题，传入12月的默认时间时，会选中到1月

HeJingWei commented 2 years ago

貌似是因为mMonthSpinner设置max和min的时候 月份没有加1 传入的月份是12时 12大于了11造成的

loper7 commented 2 years ago

貌似是因为mMonthSpinner设置max和min的时候 月份没有加1 传入的月份是12时 12大于了11造成的

确实是这样，之前对最大最小限制逻辑做了梳理重构，遗漏掉了

HeJingWei commented 2 years ago

貌似是因为mMonthSpinner设置max和min的时候 月份没有加1 传入的月份是12时 12大于了11造成的

确实是这样，之前对最大最小限制逻辑做了梳理重构，遗漏掉了

我将月份的MAX和MIN都+1了，这样应该不会有其他问题吧？

loper7 commented 2 years ago

我将月份的MAX和MIN都+1了，这样应该不会有其他问题吧？ 没问题了，0.5.4 做了这个处理