-
Notifications
You must be signed in to change notification settings - Fork 31
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
Resolve lineheight units warning #153
Conversation
@@ -38,6 +38,7 @@ export default StyleSheet.create({ | |||
color: COLORS.GREY, | |||
fontSize: 20, | |||
lineHeight: 26, | |||
padding: 1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this correct @kajsaunge ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh thanks, I'll remove it 😬
}, | ||
const SafeAreaViewWrapper = styled(SafeAreaView)({flex: 1}); | ||
|
||
const ScrollViewWrapper = styled(ScrollView)({flex: 1}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You actually don't need to import components from react-native
. Here you can instead write styled.ScrollView({flex: 1});
…9ki/29k into remove-stylesheet-and-fix-unit-warning
Removes usage of stylesheet create where possible
Removes style objects to use styled components
Adds px units for lineheight to get rid of units warning
Closes #143 #104