After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 750075 - Email from a specific sender not displaying right
Email from a specific sender not displaying right
Status: RESOLVED FIXED
Product: geary
Classification: Other
Component: general
0.10.x
Other Linux
: Normal normal
: 0.12.0
Assigned To: Geary Maintainers
Geary Maintainers
Depends on: geary-wk2 765516
Blocks:
 
 
Reported: 2015-05-29 00:51 UTC by Vincent B
Modified: 2017-02-07 13:32 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Mail example not displaying correctly (14.29 KB, text/plain)
2015-05-29 00:51 UTC, Vincent B
  Details
Don't allow body-element surrogate to set height (1.79 KB, patch)
2015-05-29 16:39 UTC, Robert Schroll
none Details | Review

Description Vincent B 2015-05-29 00:51:17 UTC
Created attachment 304206 [details]
Mail example not displaying correctly

All the messages I receive from that Sender display only the line "If you don't see that email correctly"

or sometimes only a small portion of an image when I click "Display Pictures"

Thanks for the really good work! ^-^
Comment 1 Robert Schroll 2015-05-29 14:51:24 UTC
The problem is that the email has some silly HTML.  Specifically, the body element is styled with "height: 23px".  They obviously don't want a 23-pixel tall body, and they won't get it in most cases, because browsers ignore attempts to set the height of the body element.

However, since we display multiple emails in the same HTML page in the conversation viewer, we have to turn that body element into a div, which does respect the height styling.  This is why you get a 23-pixel tall body to the email.

I note that the GMail web interface, which must do similar things, changes "height" to "min-height" on the element standing in for the body.  We could also just add a "height: auto !important" rule to our CSS.
Comment 2 Robert Schroll 2015-05-29 16:39:55 UTC
Created attachment 304276 [details] [review]
Don't allow body-element surrogate to set height

Forcing height: auto was easier than finding, parsing, and correcting CSS rules, so that's what this patch does.
Comment 3 Michael Gratton 2016-04-12 06:24:04 UTC
I'm going to be setting min-height: 100% on the body in Bug 728002, so this should be fixed with that.
Comment 4 Michael Gratton 2016-10-07 13:41:25 UTC
Well, this may have been fixed on master by 898fa33, as part of Bug 728002, but I'm not sure. There will likely be some more changes to Geary's internal HTML CSS for Bug 728002, so this will need to wait for that as well.
Comment 5 Michael Gratton 2017-02-07 13:32:30 UTC
Fix pushed to master as commit fdde9b1.