-
Notifications
You must be signed in to change notification settings - Fork 4
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
Updated How It Works #147
Updated How It Works #147
Conversation
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.
@@ -9,7 +9,7 @@ export default function ParticipatingStores({ participating, guest }) { | |||
const navigation = useNavigation(); | |||
return ( | |||
<ColumnContainer | |||
style={guest ? { marginLeft: 16, marginBottom: 24 } : { marginTop: 28 }}> | |||
style={guest ? { marginLeft: 16, marginBottom: 40 } : { marginTop: 28 }}> | |||
<Overline style={{}}>Participating Stores</Overline> |
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.
These empty style brackets??
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.
components/rewards/RewardsHome.js
Outdated
}} | ||
/> | ||
</View> | ||
<ParticipatingStores participating={participating} guest /> |
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.
Wait this should not include guest
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.
🚢
What's new in this PR
Used Ace's new How It Works photos in place of the old one! Added a third tab to make it more convenient for users to resolve #137 .
How to review
See if the styling holds up, and if spacing looks alright. Haven't tested on smaller devices as well, so please double check.
Relevant Links
Online sources
Related PRs
Next steps
Screenshots
//: # "Add screenshots of expected behavior - GIFs if you're feeling fancy!"
CC: @anniero98 @wangannie