Skip to content

Conversation

@gh-clement-taboulot
Copy link
Contributor

New section that explain how to create a bottom form.

@coveralls
Copy link

coveralls commented May 3, 2018

Pull Request Test Coverage Report for Build 68

  • 0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.9%) to 55.45%

Totals Coverage Status
Change from base Build 59: 0.9%
Covered Lines: 85
Relevant Lines: 135

💛 - Coveralls

Copy link
Contributor

@Almouro Almouro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Just small syntax comments, thanks for the PR!

README.md Outdated
/>
);

name="gender"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this should be here :D

README.md Outdated
);
```

## Troubleshooting
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would rename it Guide

README.md Outdated

## Troubleshooting

### Create a form at the bottom of a page
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would rename to "Move form above keyboard"

README.md Outdated

### Create a form at the bottom of a page

The purpose of this section is to give you a solution to create a bottom form which will go up when the keyboard appear, and the content at the top at the page will disappear.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

appears

README.md Outdated

The purpose of this section is to give you a solution to create a bottom form which will go up when the keyboard appear, and the content at the top at the page will disappear.

You have to :
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no space before : ;)

README.md Outdated

You have to :

* Create a form like you learn before ;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

; -> : no?
learnt

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's more "like you learnt above"

README.md Outdated
You have to :

* Create a form like you learn before ;
* Use [react-native-keyboard-spacer](https://github.com/Andr3wHur5t/react-native-keyboard-spacer) : it will create view with the keyboard's size when the keyboard will opened;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no space before :

README.md Outdated

* Create a form like you learn before ;
* Use [react-native-keyboard-spacer](https://github.com/Andr3wHur5t/react-native-keyboard-spacer) : it will create view with the keyboard's size when the keyboard will opened;
* Use [react-native-hide-with-keyboard](https://github.com/bamlab/react-native-hide-with-keyboard) : it will hide component when the keyboard will opened.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no space before :


Enjoy your life :

![iOS](https://github.com/bamlab/react-native-formik/blob/bottom-form/doc/images/bottomForm.gif)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! :D

export default AdoptACat;
```

For Android, we don't have to use react-native-keyboard-spacer because `android:windowSoftInputMode` is in `adjustResize` mode. Indeed, the view is automatically resize and you don't have to fill it like on iOS.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Review's changes
@Almouro Almouro merged commit 997bf02 into master May 16, 2018
@Almouro Almouro deleted the bottom-form branch May 16, 2018 08:21
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.

4 participants