-
Notifications
You must be signed in to change notification settings - Fork 12
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
Downstream changes from Explorer UX upgrades #48
Conversation
Just little nitpicking that it's not camel-case but simply Title Case |
maxWidth: [768, 1024, 1280] | ||
} | ||
|
||
export default Container |
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.
This is cool. I like!
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.
I put very little thought into what the values should be. They sound reasonable, we need to make sure they look appropriate too.
"jest-styled-components": "^6.2.0", | ||
"react-test-renderer": "^16.5.2" | ||
"jest": "^24.3.1", | ||
"jest-styled-components": "^6.3.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.
@sarathms let's not forget to file a ticket about jest-styled-components and it blocking on us upgrading to styled-components v4.
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.
Filed in #49
I left a couple of comments on this PR, but it looks good to me and I would say it can be merged. @sarathms do you need permissions to push to npm or do you have those already? |
I have published this package on npm before. I will go ahead. |
OONI Explorer is going through a major UX overhaul. There are some elements of the design system which are being redesigned. These will eventually land in other applications using them.