#31719 closed defect (bug) (fixed)
Gmail doesn't obey embedded emoji height CSS
Reported by: | pento | Owned by: | pento |
---|---|---|---|
Milestone: | 4.2 | Priority: | normal |
Severity: | normal | Version: | 4.2 |
Component: | Emoji | Keywords: | emoji good-first-bug has-patch commit |
Focuses: | Cc: |
Description
When an <img>
has an inline CSS rule using height
, it's transformed info min-height
. This means that we get massive smilies and emoji in emails.
The fix is to change height
to max-height
.
Attachments (3)
Change History (16)
#4
@
10 years ago
- Keywords has-patch added; needs-patch removed
I attached a patch that should fix this, could you guys test this? (my core install doesn't seem to be sending email for some reason so I couldn't test it myself)
#6
@
10 years ago
- Keywords needs-refresh added
Thanks for the patch, @janhenckens!
Instead of the CSS block, we need to change the inline style defined on the emoji and smiley <img>
tags. There are two defined in wp_staticize_emoji()
, and one in translate_smiley()
, both in formatting.php
.
#7
@
10 years ago
Hey @pento, thanks for the pointer!
I added a second patch that fixes the inline styles, could you give that a try? Thanks!
#8
@
10 years ago
Hey @janhenckens, there's a second <img>
tag in wp_staticize_emoji()
, could you fix that one up, too?
I just ran into this issue and solved it by setting max-height which it doesn't mess with.