'Re: Review Request 106118: Plasmate: Add KWin Scripting support to konsolepreviewer.'

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

List:       kde-panel-devel
Subject:    Re: Review Request 106118: Plasmate: Add KWin Scripting support to konsolepreviewer.
From:       "Antonis Tsiapaliokas" <kok3rs () gmail ! com>
Date:       2014-03-17 12:29:20
Message-ID: 20140317122920.13830.55098 () 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/106118/
-----------------------------------------------------------

(Updated March 17, 2014, 12:29 p.m.)

Status
------

This change has been marked as submitted.

Review request for kwin, Plasma, Martin Gräßlin, Sebastian Kügler, and Giorgos \
Tsiapaliokas.

Repository: plasmate

Description
-------

Hello,

This patch is adding the konsolepreviewer support to the KWin Scripts.
When the user, is pressing the "Execute" button, then the script is being called.
Also on the toolbar, there is a new action with name "konsole". This action is \
visible only when we are inside to a KWin Script. All the other packages, have a \
previewer. So the "konsole" action is located inside the previewer.

Diffs
-----

  plasmate/konsole/konsolepreviewer.h 8dd4369 
  plasmate/konsole/konsolepreviewer.cpp 7e180cb 
  plasmate/mainwindow.h 4daaaea 
  plasmate/mainwindow.cpp 7a96925 

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

Testing
-------

Screenshots
-----------

konsole
  https://git.reviewboard.kde.org/r/106118/s/693/

Thanks,

Antonis Tsiapaliokas

[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/106118/">https://git.reviewboard.kde.org/r/106118/</a>
  </td>
    </tr>
   </table>
   <br />

<table bgcolor="#e0e0e0" width="100%" cellpadding="8" style="border: 1px gray \
solid;">  <tr>
  <td>
   <h1 style="margin-right: 0.2em; padding: 0; font-size: 10pt;">This change has been \
marked as submitted.</h1>  </td>
 </tr>
</table>
<br />

<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, Plasma, Martin Gräßlin, Sebastian Kügler, and \
Giorgos Tsiapaliokas.</div> <div>By Antonis Tsiapaliokas.</div>

<p style="color: grey;"><i>Updated March 17, 2014, 12:29 p.m.</i></p>

<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
plasmate
</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;">Hello,

This patch is adding the konsolepreviewer support to the KWin Scripts.
When the user, is pressing the &quot;Execute&quot; button, then the script is being \
called. Also on the toolbar, there is a new action with name &quot;konsole&quot;. \
This action is visible only when we are inside to a KWin Script. All the other \
packages, have a previewer. So the &quot;konsole&quot; action is located inside the \
previewer.

</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>plasmate/konsole/konsolepreviewer.h <span style="color: \
grey">(8dd4369)</span></li>

 <li>plasmate/konsole/konsolepreviewer.cpp <span style="color: \
grey">(7e180cb)</span></li>

 <li>plasmate/mainwindow.h <span style="color: grey">(4daaaea)</span></li>

 <li>plasmate/mainwindow.cpp <span style="color: grey">(7a96925)</span></li>

</ul>

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

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>

<div>

 <a href="https://git.reviewboard.kde.org/r/106118/s/693/"><img \
src="https://git.reviewboard.kde.org/media/uploaded/images/2012/08/22/kwin_script_konsole_previewer_400x100.png" \
style="border: 1px black solid;" alt="konsole" /></a>

</div>

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

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

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

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