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 692005 - Changing character encoding doesn't work
Changing character encoding doesn't work
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
3.6.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
evolution[webkit]
Depends on:
Blocks:
 
 
Reported: 2013-01-18 11:44 UTC by Tomas Bzatek
Modified: 2013-01-18 17:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
mbox file (3.29 KB, text/plain)
2013-01-18 11:44 UTC, Tomas Bzatek
  Details
evo patch (17.72 KB, patch)
2013-01-18 17:04 UTC, Milan Crha
committed Details | Review

Description Tomas Bzatek 2013-01-18 11:44:00 UTC
Created attachment 233747 [details]
mbox file

(related to bug 692003)

Trying to change encoding in mail preview (either in main Evolution window or even opened in separate one) does nothing. Not sure if it's related to a malformed mail which pretends to be HTML (no plaintext) but misses an encoding header.

Clicking the View -> Character encoding menu does nothing, still getting question marks for extended characters. Displaying other mail and returning back has no effect, so no a refresh issue.
Comment 1 Milan Crha 2013-01-18 13:45:49 UTC
Confirming, I see it too. Probably something broke with the webkit change.
Comment 2 Milan Crha 2013-01-18 17:04:49 UTC
Created attachment 233774 [details] [review]
evo patch

for evolution;

This was tricky, the all code is there, and apparently working. The first tricky part was that the EMailDisplay has set the charset properly on its EMailFormatter instance, but this formatter is not used for formatting, because the EMailRequest creates its own vanilla instance, thus the tricky part is to pass the charset settings into the EMailRequest. The second tricky part was to pass the charset settings into the EMailPrinter. It's doable, but means a little bit of code duplication.
Comment 3 Milan Crha 2013-01-18 17:36:50 UTC
Created commit 722a27e in evo master (3.7.5+)
Created commit 45aeb9e in evo gnome-3-6 (3.6.3+)