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: 🐛 consider initial rangeStartDay value in range selection #875

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

moazelsawaf
Copy link

Hello there 👋🏻,

I just wanted to thank you for this great package! It's been really helpful for my Flutter projects 💪🏻💙

While using it in one of my apps, I found a small bug. It wasn't considering the starting day I set in rangeStartDay parameter in Range Selection. So, when I tapped on a day, it started selecting from the Start Day instead of just the End Day like it should 🐛

In this PR, I fixed the bug, tested it well, and now it works perfectly without affecting another functionality in the package 🛠️👍🏻

I hope you find it helpful 🙏🏻
Thanks again for your hard work in the Flutter community 💙 🙏🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant