D22379: Don't use deprecated QDesktopWidget

[Vlad Zagorodniy]

zzag added inline comments.

INLINE COMMENTS

> davidedmundson wrote in monitor.cpp:54
> I'm not really sure it's that important. You have one preview, but the thing you're affecting changes all N screens.
> 
> Showing dimensions of the primary screen is just as valid.
> 
> Also the previous use of availableGeometry is almost certainly wrong. It's a preview of a monitor, therefore why exclude panels?

> I'm not really sure it's that important. You have one preview, but the thing you're affecting changes all N screens.

Hmm, I just wanted to preserve current behavior. Am I missing something?

> It's a preview of a monitor, therefore why exclude panels?

I have no idea, but yeah it doesn't make sense given that the monitor preview reflects dimensions of the screen on which the kcm is shown.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D22379

To: zzag, #kwin
Cc: davidedmundson, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190710/08430698/attachment.html>