-
Notifications
You must be signed in to change notification settings - Fork 27
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
new preview + share pane #591
Conversation
_ = require('lodash'), | ||
previewSizes = { | ||
small: { w: 375, h: 660 }, | ||
medium: { w: 768, h: 1024 }, |
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 think our "medium" screen size in clay is 600? That's what we base our css off of at least
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.
our breakpoint is 600 (for the most part), but I think the most common "tablet-ish" screen width is 768
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.
That's what everyone tells me but it seems like we use them both arbitrarily :\ Another thing on the list of stuff I'd like to standardize across design + dev.
👍 |
1 similar comment
👍 |
Preview Actions
Share Actions