Plasma 5.2 bits for kdereview

[Albert Astals Cid]

El Dimarts, 23 de desembre de 2014, a les 13:17:46, Daniel Vrátil va escriure:
> On Friday, December 19, 2014 06:46:11 PM Luigi Toscano wrote:
> > Jonathan Riddell ha scritto:
> > > Plasma 5.2 is due out next month and there's a few KDE projects which
> > > would be good to be included.  Please review these for inclusion in
> > > kde/workspace..
> > > 
> > > 
> > > kscreen and libkscreen maintained by Dan Vrátil.  libkscreen is already
> > > released with Plasma but isn't in kde/workspace.
> > > 
> > >  https://projects.kde.org/projects/extragear/libs/libkscreen
> > >  https://projects.kde.org/projects/extragear/base/kscreen
> > 
> > I disagree with moving libkscreen to kde/workspace. It is a dependency for
> > at least one application (Okular), which has no Framework version for now
> > but it will have it. It would make more sense to have libkscreen in
> > Frameworks, like libnm*
> 
> AFAIK Okular is using KScreen only to get DPI info, which was not provided
> by QScreen in Qt 4. In Qt 5 you already have DPI API in QScreen, so you
> should not need KScreen anymore.

It is provided by QX11Info in Qt4, it's just wrong, does QScreen provide it 
correctly like kscreen does?

Cheers,
  Albert

> 
> 
> Dan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20150101/225e117a/attachment.sig>