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.

Sign up for GitHub

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

Jump to bottom

Refresh UI newsletter tile #126

Closed
sofiaromorales opened this issue Dec 4, 2020 · 3 comments
Closed

Refresh UI newsletter tile #126

sofiaromorales opened this issue Dec 4, 2020 · 3 comments
Assignees
@sofiaromorales

Comments

@sofiaromorales
Copy link
Member

Screen Shot 2020-12-03 at 11 42 49 PM
@sofiaromorales sofiaromorales self-assigned this Dec 4, 2020
@sofiaromorales
Copy link
Member Author

Screen Shot 2020-12-04 at 12 37 54 AM

@otech47 any recommendation here?

sofiaromorales added a commit that referenced this issue Dec 4, 2020
@sofiaromorales
closes #126
f88219c
@otech47
Copy link
Member

otech47 commented Dec 4, 2020

Screen Shot 2020-12-04 at 12 37 54 AM

@otech47 any recommendation here?

@sofiarm21 yeah I think it looks good! Maybe a tiny bit more spacing between tiles and make sure they have cursor: pointer to appear clickable since they look like material cards and that's a pretty standard UIUX pattern then probably good to merge!

@otech47
Copy link
Member

otech47 commented Dec 4, 2020

Duplicate/merge with #114

@otech47 otech47 closed this as completed Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sofiaromorales sofiaromorales

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@otech47 @sofiaromorales