<table><tr><td style="">dhaumann 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/D18083">View Revision</a></tr></table><br /><div><div><p>Patch looks interesting, didn't know about <a href="http://doc.qt.io/qt-5/qlineedit.html#addAction" class="remarkup-link" target="_blank" rel="noreferrer">http://doc.qt.io/qt-5/qlineedit.html#addAction</a> eith TrailingPosition.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D18083#inline-100603">View Inline</a><span style="color: #4b4d51; font-weight: bold;">plugin_search.cpp:97</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">actionPointers</span> <span style="color: #aa2211"><<</span> <span class="n">menuEntry</span><span class="p">(</span><span class="n">menu</span><span class="p">,</span> <span class="n">QStringLiteral</span><span class="p">(</span><span style="color: #766510">"</span><span style="color: #bb6622">\\</span><span style="color: #766510">n"</span><span class="p">),</span> <span class="n">QStringLiteral</span><span class="p">(</span><span style="color: #766510">""</span><span class="p">),</span> <span class="n">i18n</span><span class="p">(</span><span style="color: #766510">"Line break"</span><span class="p">));</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="n">actionPointers</span> <span style="color: #aa2211"><<</span> <span class="n">menuEntry</span><span class="p">(</span><span class="n">menu</span><span class="p">,</span> <span class="n">QStringLiteral</span><span class="p">(</span><span style="color: #766510">"</span><span style="color: #bb6622">\\</span><span style="color: #766510">t"</span><span class="p">),</span> <span class="n">QStringLiteral</span><span class="p">(</span><span style="color: #766510">""</span><span class="p">),</span> <span class="n">i18n</span><span class="p">(</span><span style="color: #766510">"Tab"</span><span class="p">));</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">An empty QStringLiteral should always be a simple QString().</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D18083#inline-100595">View Inline</a><span style="color: #4b4d51; font-weight: bold;">sars</span> wrote in <span style="color: #4b4d51; font-weight: bold;">plugin_search.cpp:461</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Note that this icon is not available with gnome icon theme. you need to specify that Kate depends on Breeze icon theme and actually have it installed to get this icon...</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">A solution is to use the overload with a fallback: if the icon is not found, use a more generic one that at least exists. See:<br />
<a href="http://doc.qt.io/qt-5/qicon.html#fromTheme-1" class="remarkup-link" target="_blank" rel="noreferrer">http://doc.qt.io/qt-5/qicon.html#fromTheme-1</a></p>

<p style="padding: 0; margin: 8px;">More so, the concept of fallback themes was introduced for 5.12 to address this as well, see: <a href="http://doc.qt.io/qt-5/qicon.html#setFallbackThemeName" class="remarkup-link" target="_blank" rel="noreferrer">http://doc.qt.io/qt-5/qicon.html#setFallbackThemeName</a></p>

<p style="padding: 0; margin: 8px;">Probably not applicable here, though.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18083">https://phabricator.kde.org/D18083</a></div></div><br /><div><strong>To: </strong>gregormi, Kate<br /><strong>Cc: </strong>dhaumann, loh.tar, sars, kwrite-devel, hase, michaelh, ngraham, demsking, cullmann<br /></div>