There's no proper replacement for KIcon

[Aleix Pol]

On Wed, Sep 3, 2014 at 10:01 PM, Albert Astals Cid <aacid at kde.org> wrote:

> El Dimecres, 3 de setembre de 2014, a les 21:42:24, Mark Gaiser va
> escriure:
> > On Wed, Sep 3, 2014 at 7:16 PM, Albert Astals Cid <aacid at kde.org> wrote:
> > > Porting guide lies in saying that KIcon should be replaced with
> > > QIcon::fromTheme since KIcon defauted to oxygen icon theme and
> obviously
> > > QIcon::fromTheme doesn't do that.
> > >
> > > Please someone provide a solution for this problem or undeprecate
> KIcon.
> > > This is blocking the porting of my applications.
> >
> > I've seen that issue before.
> > The thing that fixed it for me was setting the environment variable:
> > export KDE_FULL_SESSION=true
> >
> > I hope it still works :)
>
> That's not a fix, that's a workaround, we can't ask all the users out
> there to
> export that variable.
>
> Cheers,
>   Albert


It's a workaround only for non-Plasma setups, but agreed.

What about documenting that if Oxygen is a hard dependency then a call to
QIcon::setThemeSearchPaths should be made? We could even use KIconThemes
for figuring out what's the Oxygen path.

Aleix
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140904/de9685fa/attachment.html>