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 > > > > > > > > > > > > > > > > > > > > > > > >