jdtcn / BlazorDateRangePicker

A Blazor component for choosing date ranges and dates
MIT License
182 stars 35 forks source link

Calendar's wired behaviour while we have both binding and limits #69

Closed venkatesh-thatham closed 2 years ago

venkatesh-thatham commented 2 years ago

I'm using @bind-StartDate="ViewModel.Model.CheckIn" @bind-EndDate="ViewModel.Model.CheckOut" to set default date selection

Default Values CheckIn = new DateTime(DateTime.Now.Year, DateTime.Now.Month, DateTime.Now.AddDays(1).Day), CheckOut = new DateTime(DateTime.Now.Year, DateTime.Now.Month, DateTime.Now.AddDays(7).Day)

and also I set limits so then our users unable to choose beyond provisioned limits

Limits MinDate="DateTimeOffset.Now.AddDays(1)" MaxDate="DateTimeOffset.Now.AddYears(1)"

image

above image is the response of the calendar.

jdtcn commented 2 years ago

Based on the parameters, everything seems to be correct. What behavior do you expect?

venkatesh-thatham commented 2 years ago

If you notice the calendar the month is duplicated. Expected behaviour is calendar must show Dec and Jan, not Dec and Dec.

On Fri, 17 Dec, 2021, 19:53 Sergey, @.***> wrote:

Based on the parameters, everything seems to be correct. What behavior do you expect?

— Reply to this email directly, view it on GitHub https://github.com/jdtcn/BlazorDateRangePicker/issues/69#issuecomment-996664655, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIKPXNSOT3WBX65UTLFV2MDURMQBXANCNFSM5KGGX3JQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

jdtcn commented 2 years ago

Hello,

I fixed this issue in version 4.1.0, but please note that your code CheckOut = new DateTime(DateTime.Now.Year, DateTime.Now.Month, DateTime.Now.AddDays(7).Day) will not work correctly at the end of December, when there are less than 7 days before the new year. Just use CheckOut = DateTime.Now.AddDays(7) and there will be no problem

venkatesh-thatham commented 2 years ago

Thank you for your response

On Thu, 30 Dec, 2021, 01:41 Sergey, @.***> wrote:

Closed #69 https://github.com/jdtcn/BlazorDateRangePicker/issues/69.

— Reply to this email directly, view it on GitHub https://github.com/jdtcn/BlazorDateRangePicker/issues/69#event-5825592897, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIKPXNRFPXOCMJD5LMNNWV3UTNB6JANCNFSM5KGGX3JQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>