Bug 374260 - KMail 5.3.2 - Printing via Browser does not respect locale
Summary: KMail 5.3.2 - Printing via Browser does not respect locale
Status: RESOLVED FIXED
Alias: None
Product: kmail2
Classification: Applications
Component: general (show other bugs)
Version: 5.3.2
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-12-28 17:12 UTC by Axel Braun
Modified: 2017-08-09 07:28 UTC (History)
2 users (show)

See Also:
Latest Commit: https://commits.kde.org/messagelib/8f9f8d69d7ed3fa70e6af4ddd445daaf644c007a
Version Fixed In: 5.6.0

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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