D15275: [effects/magnifier] Fix 1 px gap between magnified area and frame
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
zzag added a comment. In D15275#323330 <https://phabricator.kde.org/D15275#323330>, @davidedmundson wrote: > I struggle to see how > > QRectF areaF = area; > inside the one block where we make the vertices is a "can of worms". It's scarcely even one worm. > > but you have this approved, do whichever. Well, we have area and areaF, which is not really clean. It would be cleaner to use type of QRectF for area, but then things become ugly. Anyway, I'll do your way. REPOSITORY R108 KWin BRANCH effects-magnifier-fix-1px-gaps REVISION DETAIL https://phabricator.kde.org/D15275 To: zzag, #kwin, davidedmundson Cc: davidedmundson, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart