'Re: Review Request 127985: [effects] Do not try to opacify highlighted window'

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

List:       kwin
Subject:    Re: Review Request 127985: [effects] Do not try to opacify highlighted window
From:       Anthony Fieroni <bvbfan () abv ! bg>
Date:       2016-06-01 10:18:33
Message-ID: 20160601101833.29452.11680 () mimi ! kde ! org
[Download RAW message or body]

--===============4146443186290656348==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127985/
-----------------------------------------------------------

(Updated Юни 1, 2016, 1:18 след обяд)

Review request for kwin, Martin Gräßlin and Thomas Lübking.

Repository: kwin

Description
-------

It's strange to me, why it is performed.

Diffs (updated)
-----

  deleted.h b89c961 
  deleted.cpp 5e0225f 
  libkwineffects/kwineffects.cpp 206afa3 

Diff: https://git.reviewboard.kde.org/r/127985/diff/

Testing
-------

1. Enable both tooltips and highlighted windows in taskmanager settings
2. Hover task in taskmanager
3. When tooltip appear move mouse cursor over it
4. Effect is stopped
5. Now move mouse cursor back to task item or over thumbnail window
6. Tooltip is opacified causing binking it
7. Make this move faster => annoying blinking tooltip

Thanks,

Anthony Fieroni

--===============4146443186290656348==
MIME-Version: 1.0
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 8bit

<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 \
solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">  \
<tr>  <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/127985/">https://git.reviewboard.kde.org/r/127985/</a>
  </td>
    </tr>
   </table>
   <br />

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: \
1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; \
-webkit-border-radius: 6px;">  <tr>
  <td>

<div>Review request for kwin, Martin Gräßlin and Thomas Lübking.</div>
<div>By Anthony Fieroni.</div>

<p style="color: grey;"><i>Updated Юни 1, 2016, 1:18 след обяд</i></p>

<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kwin
</div>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">It's strange to me, why it is performed.</p></pre>  \
</td>  </tr>
</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;"><ol style="padding: 0;text-rendering: inherit;margin: 0 0 0 \
2em;line-height: inherit;white-space: normal;"> <li style="padding: 0;text-rendering: \
inherit;margin: 0;line-height: inherit;white-space: normal;">Enable both tooltips and \
highlighted windows in taskmanager settings</li> <li style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Hover \
task in taskmanager</li> <li style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: normal;">When tooltip appear move mouse cursor \
over it</li> <li style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: normal;">Effect is stopped</li> <li style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Now \
move mouse cursor back to task item or over thumbnail window</li> <li style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
normal;">Tooltip is opacified causing binking it</li> <li style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Make \
this move faster =&gt; annoying blinking tooltip</li> </ol></pre>
  </td>
 </tr>
</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> \
(updated)</h1> <ul style="margin-left: 3em; padding-left: 0;">

 <li>deleted.h <span style="color: grey">(b89c961)</span></li>

 <li>deleted.cpp <span style="color: grey">(5e0225f)</span></li>

 <li>libkwineffects/kwineffects.cpp <span style="color: grey">(206afa3)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/127985/diff/" style="margin-left: \
3em;">View Diff</a></p>

  </td>
 </tr>
</table>

  </div>
 </body>
</html>

--===============4146443186290656348==--

[Attachment #3 (text/plain)]

_______________________________________________
kwin mailing list
kwin@kde.org
https://mail.kde.org/mailman/listinfo/kwin

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