farhoudshapouran / react-native-ui-datepicker

Customizable React Native 📅 DatePicker component for Android, iOS, and Web. It includes single, range and multiple modes and supports different locales.
https://farhoudshapouran.github.io/react-native-ui-datepicker/
MIT License
272 stars 28 forks source link

selectedColor of Wheel should be able to be set via timePickerTextStyle #55

Closed YaoHuiJi closed 1 month ago

YaoHuiJi commented 5 months ago

hi, we can't change the selectedColor of time wheel(it will always be 'black' which is not good for dark mode), I think selectedColor of wheel should be able to be set via timePickerTextStyle.

I've submitted a pull request, but I'm not sure if my changes are appropriate. Could you please review it?

Edit: It looks like I accidentally merged another pull request #51 into this one before submitting. My apologies for the mix-up. Ths is another issue which I tried to solve in pr #51 :

If the count of day rows is not 6, the bottom blank area is too much when integrate the calendar in UI that has other views surround it.

I made some changes to reduce the bottom blank area when the count of day rows of current month is not 6, but also I'm not sure if my changes are appropriate, please review it.

Or just ignore my messy pull requests, and sovle the two issues in your more appropriate way :)

farhoudshapouran commented 4 months ago

Yes you are right currently color of TimePicker text is defined inside the code and timePickerTextStyle don't change its color, we can add another property like selectedColor for time picker text color

johnsimeroth commented 3 months ago

Can this be merged in? I also would like this feature. It's a deal breaker in dark mode as-is.

farhoudshapouran commented 1 month ago

Please upgrade to the latest version I changed the time picker component. https://github.com/farhoudshapouran/react-native-ui-datepicker/commit/e7d93648680d4f10ee5aec77244d633abe128254