There's no proper replacement for KIcon

[Albert Astals Cid]

El Dijous, 4 de setembre de 2014, a les 01:04:11, Aleix Pol va escriure:
> 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? 

QIcon::setThemeSearchPaths won't help, the paths are fine, it's the theme that 
is wrong ("hicolor" or empty string depending on which Qt version).

Cheers,
  Albert


> We could even use KIconThemes
> for figuring out what's the Oxygen path.
> 
> Aleix