<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="http://reviewboard.kde.org/r/4819/">http://reviewboard.kde.org/r/4819/</a>
     </td>
    </tr>
   </table>
   <br />


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

<div>Review request for marble and Bastian Holst.</div>
<div>By hjain.</div>


<p style="color: grey;"><i>Updated 2010-08-08 22:09:26.046205</i></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">patch updated</pre>
  </td>
 </tr>
</table>


<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;">This patch has the following features:-
1) The older &#39;Sun Control&#39; dialog box is broken into new &#39;Sun Control&#39; dialog box and &#39;Time Control&#39; dialog box.
2) The &#39;Configure Marble Desktop Globe&#39; has &#39;Date and Time&#39; tab which has configuration features for time.
3) The current time is shown in status bar.
4) The icon of the Sun is shown on the Earth for zenith feature in &#39;Sun Control&#39; dialog box.
5) Toolbar action button are added for shadow, night map and zenith features(although shadow and night map toolbar buttons are disabled in this patch because of lack of appropriate icons for them).
6) The name of ExtDateTime class is changed to MarbleClock. The license of David Roberts(author of ExtDateTime class) has also been updated for this class with this permissions.</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>/trunk/KDE/kdeedu/marble/src/QtMainWindow.h <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/QtMainWindow.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/bindings/python/sip/MarbleModel.sip <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/bindings/python/sip/SunLocator.sip <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/CMakeLists.txt <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/ExtDateTime.h <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/ExtDateTime.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/LayerManager.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/MarbleClock.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/MarbleClock.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/MarbleDataFacade.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/MarbleModel.h <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/MarbleModel.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/MarbleTimeSettingsWidget.ui <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/QtMarbleConfigDialog.h <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/QtMarbleConfigDialog.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/SunControlWidget.h <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/SunControlWidget.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/SunControlWidget.ui <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/SunLocator.h <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/SunLocator.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/TimeControlWidget.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/TimeControlWidget.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/TimeControlWidget.ui <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/blendings/SunLightBlending.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/routing/AdjustNavigation.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/lib/routing/RoutingManager.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/marble.kcfg <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/marble_part.h <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/marble_part.cpp <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/marble_part.rc <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/marbleui.rc <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/plugins/render/stars/StarsPlugin.h <span style="color: grey">(1157379)</span></li>

 <li>/trunk/KDE/kdeedu/marble/src/plugins/render/stars/StarsPlugin.cpp <span style="color: grey">(1157379)</span></li>

</ul>

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




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




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