-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
fix(components/media_modal): Aspect ratio #4128
fix(components/media_modal): Aspect ratio #4128
Conversation
@@ -1281,6 +1281,10 @@ | |||
} | |||
} | |||
|
|||
.media-modal .react-swipeable-view-container { | |||
align-items: center; |
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.
Is not this line superseded by animateHeight
option?
(btw, I like align-items: center
way since height animation doesn't look good for me, at least for now)
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.
Right, but it's superseded by opacity: 0
(did not know how else to position the canvas behind the image).
@unarist That gif looks pretty cool to me? Except that it seems a part of the old image is visible on the very left? |
@unarist We could just use the old package ( |
Maybe no one cares about it except me :) |
* fix(components/media_modal): Aspect ratio * fix: Remove useless style
* fix(components/media_modal): Aspect ratio * fix: Remove useless style
A bit over my head with the CSS here. This should fix the issues mentioned by @unarist in #2889.