<table><tr><td style="">poboiko created this revision.<br />poboiko added reviewers: aacid, mart, broulik.<br />Herald added a project: Frameworks.<br />poboiko 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/D29826">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This is alternative approach to <a href="https://phabricator.kde.org/D22488" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D22488: invoke QIcon::setFallbackThemeName a bit later</a> and commit <a href="https://phabricator.kde.org/R302:421404563e87a50d0af22eb8ca09866fd89a36cc" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">4214045</a> to KIconThemes.<br />
Okular (and most - if not all - KDE apps inherit KMainWindow, so KDE apps<br />
should have breeze icons). KMainWindow ctor should be early enough so no icons<br />
are yet loaded, but late enough so QGuiApplication is already inited.</p>

<p>This should be followed by reverting commit <a href="https://phabricator.kde.org/R302:421404563e87a50d0af22eb8ca09866fd89a36cc" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">4214045</a> in KIconThemes.</p>

<p>Original problem description (by <a href="https://phabricator.kde.org/p/mart/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mart</a>):<br />
invoking QIcon::setFallbackThemeName at QCoreApplication ctor<br />
with Q_COREAPP_STARTUP_FUNCTION breaks the internal status of<br />
QIconLoader as it instantiates it before the QPlatformTheme,<br />
but QIconLoader depends from QPlatformTheme to be already instantiated<br />
otherwise it won't load correctly, thus breaking icon loading<br />
in QtQuickControls2 styles, such as Material and Fusion<br />
see <a href="https://bugreports.qt.io/browse/QTBUG-74252" class="remarkup-link" target="_blank" rel="noreferrer">https://bugreports.qt.io/browse/QTBUG-74252</a></p>

<p>CCBUG: 402172</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Don't have GTK3 QPA plugin, so cannot test it yet.<br />
I would appreciate if someone helped me with testing :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R263 KXmlGui</div></div></div><br /><div><strong>BRANCH</strong><div><div>icon-load (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29826">https://phabricator.kde.org/D29826</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kmainwindow.cpp</div></div></div><br /><div><strong>To: </strong>poboiko, aacid, mart, broulik<br /><strong>Cc: </strong>mart, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>