<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/116560/">https://git.reviewboard.kde.org/r/116560/</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;">Oh right, casting won't work because the hiearchy is 

KTabWidget
|
QWidget
|
DebugMessageView

...so when you cast currentWidget(), it actually gives you the first QWidget. Not sure if we should fix that or not (as in make DebugMessageView the actual page widget leaving out the middle QWidget completely). 

Comments?


Otherwise I think now it's perfect, thank you.</pre>
 <br />









<p>- Martin Klapetek</p>


<br />
<p>On March 3rd, 2014, 3:39 p.m. CET, mayank jha 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 Telepathy.</div>
<div>By mayank jha.</div>


<p style="color: grey;"><i>Updated March 3, 2014, 3:39 p.m.</i></p>







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


 <a href="http://bugs.kde.org/show_bug.cgi?id=303564">303564</a>


</div>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
ktp-common-internals
</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;">Adds a button Save Log file to save the log in the currentTab of the TabWidget.</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;">Runs fine!</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>tools/debugger/debug-message-view.h <span style="color: grey">(49a4b2f)</span></li>

 <li>tools/debugger/debug-message-view.cpp <span style="color: grey">(c2ead13)</span></li>

 <li>tools/debugger/main-window.h <span style="color: grey">(3876767)</span></li>

 <li>tools/debugger/main-window.cpp <span style="color: grey">(faf7c22)</span></li>

 <li>tools/debugger/main-window.ui <span style="color: grey">(0813149)</span></li>

</ul>

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



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

<ul>

 <li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/03/03/e4d604cc-f67a-482a-9c81-0de3cbf3f2ea__savebutton.png">The Button added</a></li>

 <li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/03/03/b6bb5033-dae3-4edc-bd60-f6428d9cfeda__savedialog.png">The File Dialog Box</a></li>

</ul>





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








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