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

[Android][Question] DatePicker / TimePicker without dialog #18783

Closed
3 tasks done
sidferreira opened this issue Apr 10, 2018 · 8 comments
Closed
3 tasks done

[Android][Question] DatePicker / TimePicker without dialog #18783

sidferreira opened this issue Apr 10, 2018 · 8 comments
Labels
Platform: Android Android applications. Ran Commands One of our bots successfully processed a command. Resolution: Locked This issue was locked by the bot. Type: Discussion Long running discussion. Type: Enhancement A new feature or enhancement of an existing feature.

Comments

@sidferreira
Copy link

sidferreira commented Apr 10, 2018

Hi!
I'm making this issue for a single reason: I really would like to have access to DatePicker and TimePicker without using dialog.

But before I start developing it, I would like to know from the team if they are open to this change, or if there are strong reasons to avoid this change, which would allow renaming DatePickerIOS to DatePicker and DatePickerAndroid to DatePickerDialogAndroid.

Yes, I do plan to first make it into a module, but the decision above may change priority, tests, etc.

Thanks in advance

Environment

Environment:
  OS: macOS High Sierra 10.13.4
  Node: 8.9.4
  Yarn: 1.5.1
  npm: 5.6.0
  Watchman: 4.9.0
  Xcode: Xcode 9.3 Build version 9E145
  Android Studio: 3.1 AI-173.4697961

Packages: (wanted => installed)
  react: ^16.3.0-alpha.1 => 16.3.0-alpha.2
  react-native: 0.54.2 => 0.54.2

Steps to Reproduce

None for now, as it's much more a question

Expected Behavior

An input for a team member in order to discuss/think/plan the feature

Actual Behavior

Waiting for reply, but really happy with the team's work

ps: I just wanted to avoid the No Template label.

@kelset kelset added Type: Discussion Long running discussion. 🌟Feature Request and removed 📋No Template Platform: Android Android applications. labels Apr 10, 2018
@facebook facebook deleted a comment from react-native-bot Apr 10, 2018
@facebook facebook deleted a comment from react-native-bot Apr 10, 2018
@kelset
Copy link
Contributor

kelset commented Apr 10, 2018

(ignore the bot, it's not really good in handling these kind of issues)

You should probably cc in the people who committed into those files. Or, if you need this implementation for your team, do them anyway in a fork and eventually PR them (ideally as separate PRs, one for DatePickerIOS and the other for DatePickerAndroid). It's usually easier to discuss these things with a PR with code,

@sidferreira
Copy link
Author

@kelset So your suggestion is basically: Do it anyway, create the PR and discuss it later? hahaha
Will check the other devs and tag them indeed. It's a next week thing so I have some time...

@sidferreira
Copy link
Author

@sidferreira
Copy link
Author

sidferreira commented Apr 10, 2018

FYI: Android-wise, it's basically this code I'll implement (and some API version checks to avoid errors)

DatePicker datePicker = new DatePicker(
        this.getBaseContext(), null, R.style.DatePickerSpinner);
    datePicker.setCalendarViewShown(false);
    <style name="DatePickerSpinner">
        <!-- Customize your theme here. -->
        <item name="android:datePickerMode">spinner</item>
    </style>

@sidferreira
Copy link
Author

Note to self: #12384 #10932

@kelset kelset added the Ran Commands One of our bots successfully processed a command. label Apr 10, 2018
@facebook facebook deleted a comment from react-native-bot Apr 10, 2018
@kelset
Copy link
Contributor

kelset commented Apr 10, 2018

Do it anyway, create the PR and discuss it later?

I mean, if you need it, and you need it quickly, that's surely the fastest approach.

@sidferreira
Copy link
Author

@kelset Not exactly needing it... but really want to have an Android version to DatePickerIOS's component

@hramos
Copy link
Contributor

hramos commented Jan 30, 2019

It looks like this is an enhancement request rather than a bug report, so I will go ahead and close it. There's some related discussion going on at react-native-community/discussions-and-proposals#85, which didn't exist when this issue was originally created. Can you continue any discussion over there?

@hramos hramos closed this as completed Jan 30, 2019
@hramos hramos added the Type: Enhancement A new feature or enhancement of an existing feature. label Mar 14, 2019
@facebook facebook locked as resolved and limited conversation to collaborators Jan 30, 2020
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Jan 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Platform: Android Android applications. Ran Commands One of our bots successfully processed a command. Resolution: Locked This issue was locked by the bot. Type: Discussion Long running discussion. Type: Enhancement A new feature or enhancement of an existing feature.
Projects
None yet
Development

No branches or pull requests

4 participants