-
Notifications
You must be signed in to change notification settings - Fork 16
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
Some development improvements and fixes #1
base: master
Are you sure you want to change the base?
Conversation
fredyk
commented
Nov 10, 2017
- (Added) Show long months
- (Fixed) Leap years have 29 days in February
- (Added) Use custom font
- (Added) maxDate
- (Fixed) Prevent an IndexOutOfBoundsException at onDateSelected
- (Fixed) Properly access months and days array while setting initial positions
- (Fixed) decreasing dayPos while changing to a shorter month
- (Fixed) calculate maxTextHeight with a "j" to support lowerCase month names
- (Fixed) Leap years have 29 days in February - (Added) Use custom font - (Added) maxDate - (Fixed) Prevent an IndexOutOfBoundsException at onDateSelected - (Fixed) Properly access months and days array while setting initial positions - (Fixed) decreasing dayPos while changing to a shorter month - (Fixed) calculate maxTextHeight with a "j" to support lowerCase month names
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are the only changes I would like you to do:
- In order to keep the code aligned, rename the variable "mFont" for "font" or "fontTypeface".
- Rename the setFont setter for setTypeface to keep the code aligned with the android API.
- Add brackets to if/else when generating the Typeface.
@fredyk Also, I want to thank you for opening a PR offering bug fixes and code improvement! 😄 |
I would like to kindly ask you to close this pull request in order to create another from the master branch with just bug fixes, so that I can continue developing my own version in the develop branch, making improvements without impairing yours, and creating successive pull requests. By the way, from my own experience,I would like to suggest you to create your own developing and pre-release branches (It gave me a better control over my changes ;) ). Adiós y un saludo! |
Great! I will do it so! Thanks for the advice! |