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.

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] Gif images aspect ratio on preview #20654

Merged
merged 2 commits into from
Feb 9, 2021
Merged

Conversation

tiagoevanp
Copy link
Contributor

@tiagoevanp tiagoevanp commented Feb 9, 2021

Proposed changes (including videos or screenshots)

Issue(s)

closes #20546
closes #20307

Steps to test or reproduce

Further comments

@tiagoevanp tiagoevanp requested a review from a team February 9, 2021 00:46
@ggazzo ggazzo merged commit 374281c into develop Feb 9, 2021
@ggazzo ggazzo deleted the fix/embed-aspect-ratio branch February 9, 2021 14:56
@ggazzo ggazzo added this to the 3.11.1 milestone Feb 9, 2021
@sampaiodiego sampaiodiego mentioned this pull request Feb 10, 2021
vanhoang1107 pushed a commit to vanhoang1107/Rocket.Chat that referenced this pull request Feb 19, 2021
* rocketchat/master:
  Bump version to 3.11.1
  [FIX] Livechat bridge permission checkers (RocketChat#20653)
  Fix room not being assigned to bot agent first. (RocketChat#20662)
  [FIX] Attachment download from title fixed (RocketChat#20585)
  [FIX] Gif images aspect ratio on preview (RocketChat#20654)
  [FIX] Update NPS banner when changing score (RocketChat#20611)
@sampaiodiego sampaiodiego mentioned this pull request Feb 28, 2021
@thrakks
Copy link

thrakks commented Sep 9, 2021

This still appears to be broken as of 3.18.1.

Everything is subject to that !important max-width 480px css value. It's broken so many of our webhook attachments in terms of readability where previously you could show heaps of info using up the screen space efficiently.

Edit: Looks like I'm wrong about the max-width affecting this as I use the site-inspector to modify a bunch of those css values and the gifs in my rocketchat continued having a completely wrong and skewed ratio.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants