'Re: Review Request 115298: [kwin] Add better support for Notification windows'

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

List:       kwin
Subject:    Re: Review Request 115298: [kwin] Add better support for Notification windows
From:       "Commit Hook" <null () kde ! org>
Date:       2014-01-28 6:43:37
Message-ID: 20140128064337.11809.16783 () probe ! kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]

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

This review has been submitted with commit 4c45e1ec11f9690f241ec92cf1b5e67215d566f6 \
by Martin Gräßlin to branch master.

- Commit Hook

On Jan. 27, 2014, 6:57 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115298/
> -----------------------------------------------------------
> 
> (Updated Jan. 27, 2014, 6:57 a.m.)
> 
> 
> Review request for kwin.
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> [kwin] Add better support for Notification windows
> 
> KWin starts to support the Notification window type and has an own
> layer for all notification windows. They are kept above the above
> layer but do not go over active fullscreen windows.
> 
> 
> Diffs
> -----
> 
> kwin/utils.h 4348b98bd423e4a74bb37b0592c1e55919dd6860 
> kwin/layers.cpp 21bd4399e83c2ec0a3af33fd905d196b9412c0ba 
> 
> Diff: https://git.reviewboard.kde.org/r/115298/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
> 

[Attachment #5 (text/html)]

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

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been \
submitted with commit 4c45e1ec11f9690f241ec92cf1b5e67215d566f6 by Martin Gräßlin to \
branch master.</pre>  <br />

<p>- Commit Hook</p>

<br />
<p>On January 27th, 2014, 6:57 a.m. UTC, Martin Gräßlin wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;">  <tr>
  <td>

<div>Review request for kwin.</div>
<div>By Martin Gräßlin.</div>

<p style="color: grey;"><i>Updated Jan. 27, 2014, 6:57 a.m.</i></p>

<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kde-workspace
</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;">[kwin] Add better support for Notification windows

KWin starts to support the Notification window type and has an own
layer for all notification windows. They are kept above the above
layer but do not go over active fullscreen windows.</pre>
  </td>
 </tr>
</table>

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

 <li>kwin/utils.h <span style="color: \
grey">(4348b98bd423e4a74bb37b0592c1e55919dd6860)</span></li>

 <li>kwin/layers.cpp <span style="color: \
grey">(21bd4399e83c2ec0a3af33fd905d196b9412c0ba)</span></li>

</ul>

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

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

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

_______________________________________________
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] 