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

Update global text and field styles #58

Merged
merged 17 commits into from
Jun 19, 2019

Conversation

LanesGood
Copy link
Member

@LanesGood LanesGood commented Jun 18, 2019

Multiple styling edits for text and field styles. This is an attempt to match the text and colors used throughout the app more closely to the original UI designs.

@sethvincent can you help out with some of the todos below, especially the first and last item?

TO DO:
-[ ] Implement working icons in FeatureDetailHeader for the View and Add point screens

@LanesGood LanesGood requested a review from sethvincent June 18, 2019 15:28
app/style/variables.js Outdated Show resolved Hide resolved
@sethvincent
Copy link
Contributor

sethvincent commented Jun 19, 2019

Implement working icons in FeatureDetailHeader for the View and Add point screens

Taking care of this in separate PRs

@sethvincent sethvincent self-requested a review June 19, 2019 14:51
Also remove StyledText component from Variables.

Investigate - remove styled.Text RN component from components where 
declared with no styles? Can also then remove styled-components import
@LanesGood LanesGood changed the title Update/field styles validation Update global text and field styles Jun 19, 2019
@LanesGood LanesGood marked this pull request as ready for review June 19, 2019 16:00
@sethvincent
Copy link
Contributor

Debugging on ios now and there are some small layout issues around Add a field and Additional tags that I'll fix up and then it'll be good to merge.

@LanesGood
Copy link
Member Author

@sethvincent @kamicut I've set this PR as ready to review, as its gotten a bit large and away from the original stated purpose. I'll work on validation separately, and some of the rest of the app styling separately as well.

This does still need to be tested and debugged on IOS.

@LanesGood LanesGood requested a review from kamicut June 19, 2019 16:03
@sethvincent
Copy link
Contributor

tested and made some improvements for ios/iphonex layout of the tag editor and add a field chooser.

@sethvincent sethvincent merged commit 7cf6b49 into master Jun 19, 2019
@sethvincent sethvincent deleted the update/field-styles-validation branch June 19, 2019 18:01
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.

2 participants