D8485: Fix viewport for screenshot on multiple screens

2017-10-26 Thread David Edmundson 
This revision was automatically updated to reflect the committed changes.
Closed by commit R108:9cafbb117984: Fix viewport for screenshot on multiple 
screens (authored by kpiwowarski, committed by davidedmundson).

REPOSITORY
  R108 KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8485?vs=21343&id=21393

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

AFFECTED FILES
  libkwineffects/kwinglutils.cpp

To: kpiwowarski, #kwin, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, #kwin, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D8485: Fix viewport for screenshot on multiple screens

2017-10-26 Thread Kamil Piwowarski 
kpiwowarski added a comment.


  @davidedmundson How can I commit? This is my first contribution here. Could 
you that?

REPOSITORY
  R108 KWin

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

To: kpiwowarski, #kwin, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, #kwin, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D8485: Fix viewport for screenshot on multiple screens

2017-10-25 Thread David Edmundson 
davidedmundson accepted this revision.
davidedmundson added a comment.
This revision is now accepted and ready to land.


  thanks

REPOSITORY
  R108 KWin

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

To: kpiwowarski, #kwin, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, #kwin, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D8485: Fix viewport for screenshot on multiple screens

2017-10-25 Thread Kamil Piwowarski 
kpiwowarski created this revision.
kpiwowarski added reviewers: KWin, Plasma.
kpiwowarski added a project: KWin.
Restricted Application edited projects, added Plasma; removed KWin.
Restricted Application added subscribers: kwin, plasma-devel.

REVISION SUMMARY
  https://phabricator.kde.org/D8479 fixes viewport for real hardware, but there 
is still problem with screenshots. This patch fixes that.

TEST PLAN
  - Have three monitors (like in bug 385655)
  - Press PrtSc and then Enter
  - Screenshot is ok
  
  Before: 
  F5450056: Screenshot_20171025_232900.png 

  After:
  F5450058: Screenshot_20171026_010701.png 


REPOSITORY
  R108 KWin

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

AFFECTED FILES
  libkwineffects/kwinglutils.cpp

To: kpiwowarski, #kwin, #plasma
Cc: plasma-devel, kwin, #kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart