[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kwin
Subject:    D9487: [effects/slide] Handle moving clients
From:       Vlad Zagorodniy <noreply () phabricator ! kde ! org>
Date:       2017-12-23 10:20:47
Message-ID: differential-rev-PHID-DREV-vtshsbix6qwvvkx7mnr2-req () phabricator ! kde ! org
[Download RAW message or body]

zzag created this revision.
zzag added a reviewer: KWin.
Restricted Application added a project: KWin.
Restricted Application added a subscriber: kwin.

REVISION SUMMARY
  Currently, slide effect doesn't handle a case when there
  is a moving client. This results in having window jumps.
  This commit fixes it by fixing position of the moving client
  during switching to another desktop.
  
  Depends on https://phabricator.kde.org/D9486

TEST PLAN
  - send window one desktop to the left/right using shortcuts

REPOSITORY
  R108 KWin

BRANCH
  effects/slide-moving-window

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

AFFECTED FILES
  effects/slide/slide.cpp
  effects/slide/slide.h

To: zzag, #kwin
Cc: kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, \
jensreuterberg, abetts, sebas, apol, mart


[Attachment #3 (unknown)]

<table><tr><td style="">zzag created this revision.<br />zzag added a reviewer: \
KWin.<br />Restricted Application added a project: KWin.<br />Restricted Application \
added a subscriber: kwin. </td><a style="text-decoration: none; padding: 4px 8px; \
margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: \
3px; background-color: #F7F7F9; background-image: linear-gradient(to \
bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" \
href="https://phabricator.kde.org/D9487" rel="noreferrer">View \
Revision</a></tr></table><br /><div><strong>REVISION \
SUMMARY</strong><div><p>Currently, slide effect doesn&#039;t handle a case when \
there<br /> is a moving client. This results in having window jumps.<br />
This commit fixes it by fixing position of the moving client<br />
during switching to another desktop.</p>

<p>Depends on <a href="https://phabricator.kde.org/D9486" style="background-color: \
#e7e7e7;  border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" \
rel="noreferrer">D9486</a></p></div></div><br /><div><strong>TEST \
PLAN</strong><div><ul class="remarkup-list"> <li class="remarkup-list-item">send \
window one desktop to the left/right using shortcuts</li> </ul></div></div><br \
/><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br \
/><div><strong>BRANCH</strong><div><div>effects/slide-moving-window</div></div></div><br \
/><div><strong>REVISION DETAIL</strong><div><a \
href="https://phabricator.kde.org/D9487" \
rel="noreferrer">https://phabricator.kde.org/D9487</a></div></div><br \
/><div><strong>AFFECTED FILES</strong><div><div>effects/slide/slide.cpp<br /> \
effects/slide/slide.h</div></div></div><br /><div><strong>To: </strong>zzag, KWin<br \
/><strong>Cc: </strong>kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, \
hardening, jensreuterberg, abetts, sebas, apol, mart<br /></div>



[prev in list] [next in list] [prev in thread] [next in thread]