D20621: [screenedge effect] Support "hint-stretch-borders"

zzag accepted this revision.
zzag added a comment.
This revision is now accepted and ready to land.


  Please change prefix to "[effects/screenedge]".

INLINE COMMENTS

> screenedgeeffect.cpp:347
>          p.drawPixmap(pixmapPosition, l);
> -        p.drawTiledPixmap(QRect(l.width(), pixmapPosition.y(), size.width() - l.width() - r.width(), c.height()), c);
> +        const QRect cRect(l.width(), pixmapPosition.y(), size.width() - l.width() - r.width(), c.height());
> +        if (stretchBorder) {

Could you please rename this variable? Maybe `target`? Not a serious issue but would be nice to have a bit descriptive variable names. :)

REPOSITORY
  R108 KWin

BRANCH
  gloweffectsupportstretchborderhint

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

To: kossebau, #kwin, zzag
Cc: zzag, kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart