<table><tr><td style="">nicolasfella created this revision.<br />nicolasfella added reviewers: Plasma, mart.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />nicolasfella requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18341">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Trying to assign an icon from a file to a button like in</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">import QtQuick 2.2
import QtQuick.Controls 2.4

Button {
    icon.source: "/usr/share/icons/breath/emotes/22/face-smirk.svg"
}</pre></div>

<p>results in no icon being shown. The current code interprets the filename as QString and tries to load an icon with that name from the theme, which obviously doesn't exist. This patch checks whether the icon points to a local file <br />
and if it is loads it. If not it falls back to loading it from the theme.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Above code now correctly displays a face.<br />
Ran systemsettings, checked QML KCMs' buttons for icons</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R858 Qt Quick Controls 2: Desktop Style</div></div></div><br /><div><strong>BRANCH</strong><div><div>icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18341">https://phabricator.kde.org/D18341</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugin/kquickstyleitem.cpp</div></div></div><br /><div><strong>To: </strong>nicolasfella, Plasma, mart<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>