Bug 374260

Summary: KMail 5.3.2 - Printing via Browser does not respect locale
Product: [Applications] kmail2 Reporter: Axel Braun <axel.braun>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: montel, wbauer1
Priority: NOR    
Version: 5.3.2   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: https://commits.kde.org/messagelib/8f9f8d69d7ed3fa70e6af4ddd445daaf644c007a Version Fixed In: 5.6.0
Sentry Crash Report:

Description Axel Braun 2016-12-28 17:12:33 UTC 
KMail 5.3.2 uses printing via Browser as workaround.
Unfortunately this seems not to respect the locale:
While a mail is shown correct with Umlauts (ä, ü, ö), the document in the browser shows e.g. verzögert instead of verzögert
Comment 1 Axel Braun 2017-08-04 16:29:08 UTC 
Still a problem in 5.5.3
Comment 2 Laurent Montel 2017-08-05 14:55:56 UTC 
Please use qt >= 5.8
we use real printing support no external html browser for doing it.
Comment 3 Axel Braun 2017-08-07 07:10:18 UTC 
(In reply to Laurent Montel from comment #2)
> Please use qt >= 5.8
> we use real printing support no external html browser for doing it.

Right, I tested on Leap 42.3, which uses qt 5.6.x
In current Tumbleweed it works! So its a packaging issue -> will file a bug against Leap and close this one.

Thanks Laurent!
Comment 4 Wolfgang Bauer 2017-08-08 16:40:49 UTC 
But an external html browser is still used for print preview even with Qt 5.9, and this has the same problem.

I have a fix for it though:
https://phabricator.kde.org/D7206
Comment 5 Wolfgang Bauer 2017-08-09 06:16:50 UTC 
Git commit 8f9f8d69d7ed3fa70e6af4ddd445daaf644c007a by Wolfgang Bauer.
Committed on 09/08/2017 at 06:14.
Pushed by wbauer into branch 'Applications/17.08'.

Fix specifying the charset when exporting as HTML

The QString::replace() had no effect because the stream actually
contains <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">, not just <!DOCTYPE html>.

Also, the <meta> tag should be inside the <head> section.
FIXED-IN: 17.08.0
Differential Revision: https://phabricator.kde.org/D7206

M  +1    -1    webengineviewer/src/webengineexporthtmlpagejob.cpp

https://commits.kde.org/messagelib/8f9f8d69d7ed3fa70e6af4ddd445daaf644c007a