Ah, just saw https://phabricator.kde.org/D14983 - so it's handled automatically.
Thanks & cheers Dominik On Sun, Oct 14, 2018 at 7:53 PM Dominik Haumann <dhaum...@kde.org> wrote: > > On Sat, Sep 1, 2018 at 1:09 AM Albert Astals Cid <aa...@kde.org> wrote: > > > > El dissabte, 1 de setembre de 2018, a les 1:02:00 CEST, Dominik Haumann va > > escriure: > > > ...so should we revert the hack we did in Kate? > > > > Well, this won't be available until wuite some time in the future (Qt 5.12) > > so I'd say for now leave it in. > > Btw, since this landed in Qt 5.12, do KDE apps automatically use this > via KDE's platform integration layer? > ...or is it that every application needs to set the fallback theme iteslf? > > Greetings > Dominik > > > > > Cheers, > > Albert > > > > > > > > Greetings > > > Dominik > > > > > > > > > Albert Astals Cid <aa...@kde.org> schrieb am Di., 28. Aug. 2018, 22:45: > > > > > > > El dijous, 16 d’agost de 2018, a les 15:19:36 CEST, Albert Astals Cid va > > > > escriure: > > > > > Missatge de Albert Vaca <albertv...@gmail.com> del dia dj., 16 d’ag. > > > > 2018 a > > > > > les 13:57: > > > > > > > > > > > Hi everyone, > > > > > > > > > > > > If we want to reach more potential users, we have to make sure KDE > > > > > > apps look properly in every desktop. Take a look at how KDE apps > > > > > > look > > > > > > in Gnome, you will see there are lots of missing icons. > > > > > > > > > > > > https://imgur.com/a/nkeiryb > > > > > > > > > > > > It think it is important to get this fixed, as it can make people > > > > > > not > > > > > > use our apps. > > > > > > > > > > > > Note that, on the other hand, Gnome apps look good in Plasma even > > > > > > when > > > > > > Breeze doesn't have the icons they need. This is because they fall > > > > > > back to their own theme when an icon is not found in the system > > > > > > theme: > > > > > > I just tested it setting the icon theme to Breeze on Gnome, and > > > > > > deleting the "actions" directory from Breeze, they show the Gnome > > > > > > icons instead. > > > > > > > > > > > > Anyone has a suggestion on how to fix this? > > > > > > > > > > > > > > > > If we can get this accepted into Qt > > > > > https://codereview.qt-project.org/#/c/237025/ > > > > > Which i think we could since it mimics nicely the setThemeName > > > > > functionality. > > > > > > > > > > We can then just add a Q_COREAPP_STARTUP_FUNCTION in KIconTheme that > > > > > does > > > > > QIcon::setFallbackThemeName("breeze"); > > > > > > > > > > And that'd be a one place fix all solution. > > > > > > > > FWIW this just landed > > > > https://phabricator.kde.org/D14983 > > > > > > > > Cheers, > > > > Albert > > > > > > > > > > > > > > Cheers, > > > > > Albert > > > > > > > > > > > > > > > > > > > > > > (Please CC me as I'm not subscribed to the list.) > > > > > > > > > > > > Albert > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >