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
Stories share menu redesign. #19303
Stories share menu redesign. #19303
Conversation
LGTM. Percy tests look broken though, can you take a look? |
display: none !important; | ||
} | ||
|
||
@media (min-width: 1024px) { |
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.
Can we actually bind this to the desktop/fullbleed attributes instead of using media queries?
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.
Done (for the desktop-fullscreen
only, as this menu doesn't exist on the panels experience)
3919734
to
0384c29
Compare
Percy should fail in an expected way (slightly bigger font size) this time :) |
0384c29
to
1dc80ed
Compare
* Share menu redesign. * Reviews.
Redesign of the story share menu to make it match with all our other white drawers, and more colorful. Also adding some responsive magic for when it's displayed on desktop.
The bookend is unchanged, except for the share item label that went from a font size of 10 to 11px.
#19270