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

Change the android app styles #20

Merged
merged 9 commits into from Mar 30, 2017
Merged

Change the android app styles #20

merged 9 commits into from Mar 30, 2017

Conversation

markusguenther
Copy link
Collaborator

The style of the android app is not optimal. The most text components are not centered vertically
and the input text changes the height depending on the inlineLabel property.

This commit does not remove the input lines because this is a component property of
text-input in react native and can be changed in the general behavior.

But to change this you also need the pull request 15.

#15

Fixes: #9

@markusguenther
Copy link
Collaborator Author

For multiline inputs the alignment should not be centered. Will add this soon (this evening/night)

@esbenp
Copy link
Owner

esbenp commented Mar 28, 2017

There are some conflicts here. Can you resolve those and then I will review again? :-)

@markusguenther
Copy link
Collaborator Author

Ok conflicts has been resolved :)

@tylerbuchea
Copy link
Contributor

What is the word on this? I'm having issues with the forms displaying on Android as well. Would love to have this fix merged in.

@markusguenther
Copy link
Collaborator Author

I guess Esben is on it. He already wrote me that he is busy at the moment.

Stay tuned, it will not take so much time anymore ;)

@esbenp esbenp merged commit e5e7cfa into esbenp:master Mar 30, 2017
@tylerbuchea
Copy link
Contributor

Sorry for being pushy I didn't look at how recent this PR was before I commented. Your response time is more than satisfactory. Thanks for all the hard work @esbenp and @markusguenther 😄

@markusguenther
Copy link
Collaborator Author

No problem ... your welcome :)

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.

None yet

3 participants