<table><tr><td style="">sitter added a comment.
</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/D19033">View Revision</a></tr></table><br /><div><div><p>If I understand it correctly, the bug would only show its ugly head with <tt style="background: #ebebeb; font-size: 13px;">QIcon::fromTheme("foo.svg")</tt> but not <tt style="background: #ebebeb; font-size: 13px;">QIcon::fromTheme("foo")</tt>. So, if that seems a concern we can just add a comment in the README. Distribution devs would set LogoPath's value manually, so we can just tell them to use a valid value :) Does that seem agreeable <a href="https://phabricator.kde.org/p/broulik/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@broulik</a>?</p>

<p><a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> On a side-note: <tt style="background: #ebebeb; font-size: 13px;">foo.svg</tt> is not supported by the specs, so Qt not working with it properly is not even a defect TBH - it'd be an additional convenience feature in Qt. Specifically the desktop entry spec defines <tt style="background: #ebebeb; font-size: 13px;">Icon=</tt> as either absolute or it will be run through the lookup described in the icon theme spec, and the icon theme spec sets forth a lookup which explicitly assumes <tt style="background: #ebebeb; font-size: 13px;">filename = directory/$(themename)/subdir/iconname.extension</tt> which by extension means a string handed to the theme lookup may only ever be a basename of the icon. It may not be absolute, nor relative, nor contain extensions. So, while it'd be nice if Qt handled sanitation of relative strings, our software is in fact wrong if it allows users to set a (relative) icon with file extension and then runs that through QIcon::fromTheme without adjusting it. The input must either be absolute or properly sanitized to meet the requirements of the icon theme spec.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R102 KInfoCenter</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19033">https://phabricator.kde.org/D19033</a></div></div><br /><div><strong>To: </strong>sitter, Plasma<br /><strong>Cc: </strong>ngraham, broulik, cfeck, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>