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

Use flex to automatically size optionContainer to its contents. #37

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

Conversation

peacechen
Copy link

This allows the modal contents to automatically fit any screen.

overlayStyle and cancelContainer adjusted accordingly to dynamically center content in a generic fashion.
@fabriziomoscon
Copy link

merged into my fork

@peacechen
Copy link
Author

peacechen commented Feb 2, 2017

Thanks @fabriziomoscon. Do you publish your fork to NPM?

I noticed that you also removed flex:1 from selectStyle for RN 0.39 compatibility:
fabriziomoscon@423c8a0

I haven't tested against 0.39, but if you've vetted it I'll add that change to my PR.

@fabriziomoscon
Copy link

At the moment my package.json has this:

"react-native-modal-picker": "fabriziomoscon/react-native-modal-picker",

I am not planning to republish it under a different name.

Yes I tested it with RN 0.39.2 on android

- versions of react native < 0.39.0 should pass flex:1 explicitly to selectStyle as a prop. 
- Update Readme to reflect selectStyle prop change for RN 0.39.
Proberts pushed a commit to Proberts/react-native-modal-picker that referenced this pull request Apr 13, 2017
Added changes from d-a-n#32
Added changes from d-a-n#37
Added changes from d-a-n#40
Added Properties (default):
    optionContainerStyle: ({})
    optionNumberOfLines (undefined)
    optionEllipsizeMode (tail)
@jaredtibs
Copy link

Is there an update on getting this merged? In need of this functionality but would like to use the original repo.

@fabriziomoscon
Copy link

This repo is abandoned, I have cheery picked few PR from here in my fork since I am using this code in production.

peacechen and others added 16 commits August 8, 2017 13:35
Set the container prop `pointerEvents="none"` for children (content) to allow touches to `TextInput` to flow through.
Fixes sizing when rotating between landscape & portrait orientations.
* Prop to change supported orientation
* add base eslint rules and fix lint errors
* add arbitrary eslint rules
- key-spacing
- no-empty-functions
- comma-dangle rules to prevent runtime errors
Backwards compatible with RN pre-0.44.
* add support for configurable animationType

* updated documentation
Update bitmap link and improve sample code.
tetianastepova and others added 29 commits August 19, 2020 14:32
Co-authored-by: HP <himanshu.patil@bizotics.com>
Co-authored-by: Peace <github@ghpc.33mail.com>
Co-authored-by: Phil Taylor <phil@customd.com>
See #168 for description of error
* add option param to onModalClose type

* export props
* update

1) updated expo version of sample app to 40.0.0
2) added typescript to sample app
3) converted sample app.tsx from class component to functional
4) fixed types issue

* added more types
Co-authored-by: Arthur Arakelyan <artur.araqelyan.2004@gmail.com>
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.