#31719 closed defect (bug) (fixed)
Gmail doesn't obey embedded emoji height CSS
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| 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
@
11 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
@
11 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
@
11 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
@
11 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.