<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/117277/">https://git.reviewboard.kde.org/r/117277/</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 d1e542627f907c9512859c0f002210d9bf8b221b by Alex Merry to branch master.</pre>

 <br />

<p>- Commit Hook</p>

<br />

<p>On April 1st, 2014, 10:35 a.m. UTC, Alex Merry 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 KDE Frameworks and Christoph Feck.</div>

<div>By Alex Merry.</div>

<p style="color: grey;"><i>Updated April 1, 2014, 10:35 a.m.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

kwidgetsaddons

</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;">KDatePicker: override changeEvent() instead of setEnabled()

As the comment noted, setEnabled() is not virtual.  Also, there was no

need to separately enabled/disable the individual components, when they

are children of the KDatePicker widget.  Instead, we override

changeEvent() and use it to focus the table (as before).

This also extends the kdatepicktest to temporarily disable the picker.</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;">Builds.  Tests pass.  Played around with kdatepicktest; behaviour seems unchanged.</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>src/kdatepicker.h <span style="color: grey">(855aeb01ff457394cfbcff57845a57a9eae0a103)</span></li>

 <li>src/kdatepicker.cpp <span style="color: grey">(345ee2d97632647789e6762e5e155e75802c2191)</span></li>

 <li>tests/kdatepicktest.cpp <span style="color: grey">(5f29110f2335bb110b3470007a59f9cb8b6bcfee)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>