KDE apps have missing icons when not on Breeze

[Dominik Haumann]

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 <dhaumann at kde.org> wrote:
>
> On Sat, Sep 1, 2018 at 1:09 AM Albert Astals Cid <aacid at 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 <aacid at 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 <albertvaka at 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
> > > > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> >
> >
> >
> >