D22379: Don't use deprecated QDesktopWidget
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
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