Differential D6606

Support SVG too
ClosedPublic
Actions

Authored by wbauer on Jul 10 2017, 12:50 PM.

Details

Reviewers
dfaure
Commits
R313:ed5c039f2d19: Support SVG too
Summary

Konqueror's about page doesn't show icons if the breeze icon theme is used.
The reason is that khtml doesn't support SVG images.

This patch is supposed to fix it by allowing to use Qt's support in that case.

BUG: 355872

Test Plan

Start Konqueror, the startup page now displays icons.

See also the attached images

Diff Detail

Repository
R313 KHtml
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
wbauer created this revision.Jul 10 2017, 12:50 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJul 10 2017, 12:50 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
wbauer added a comment.Jul 10 2017, 12:58 PM

Here are the promised pictures:
before:

after:

I'd like to backport this to KDE4 too....

wbauer edited the summary of this revision. (Show Details)Jul 10 2017, 12:59 PM
dfaure accepted this revision.Jul 10 2017, 2:10 PM
This revision is now accepted and ready to land.Jul 10 2017, 2:10 PM
Closed by commit R313:ed5c039f2d19: Support SVG too (authored by wbauer). · Explain WhyJul 11 2017, 11:13 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 16505

View Options

src/imload/decoders/qimageioloader.cpp

Loading...