From 43cf8c8aef98b12b46044d6fdfb79eaa95bb7896 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Stefan=20Br=C3=BCns?= Date: Wed, 1 May 2019 14:39:12 +0200 Subject: [PATCH] [EPubGenerator] Fetch the application palette just once Summary: There is no reason to fetch the palette once per loop, just do it once Reviewers: #okular, ngraham Reviewed By: ngraham Subscribers: okular-devel Tags: #okular Differential Revision: https://phabricator.kde.org/D20947 --- generators/epub/converter.cpp | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/generators/epub/converter.cpp b/generators/epub/converter.cpp index 11bd8c1bb..f6cc85d5c 100644 --- a/generators/epub/converter.cpp +++ b/generators/epub/converter.cpp @@ -201,6 +201,14 @@ QTextDocument* Converter::convert( const QString &fileName ) bool firstPage = true; QVector movieAnnots; QVector soundActions; + + // HACK BEGIN Get the links without CSS to be blue + // Remove if Qt ever gets fixed and the code in textdocumentgenerator.cpp works + const QPalette orig = qApp->palette(); + QPalette p = orig; + p.setColor(QPalette::Link, Qt::blue); + // HACK END + const QSize videoSize(320, 240); do{ if(!epub_it_get_curr(it)) { @@ -289,11 +297,7 @@ QTextDocument* Converter::convert( const QString &fileName ) htmlContent = dom.toString(); } - // HACK BEGIN Get the links without CSS to be blue - // Remove if Qt ever gets fixed and the code in textdocumentgenerator.cpp works - const QPalette orig = qApp->palette(); - QPalette p = orig; - p.setColor(QPalette::Link, Qt::blue); + // HACK BEGIN qApp->setPalette(p); // HACK END