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



 <p>Ship it!</p>



 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ship It!</pre>
 <br />







<p>- Dennis</p>


<br />
<p>On October 30th, 2012, 8:11 p.m., Utku Aydın wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/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.</div>
<div>By Utku Aydın.</div>


<p style="color: grey;"><i>Updated Oct. 30, 2012, 8:11 p.m.</i></p>






<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;">With this patch, Foursquare plugin users now can allow Marble to make requests to Foursquare on behalf of them. Currently works only on Marble Touch, and has minor problems, e.g. the login page is not scrollable.

The patch also fixes the problem with the venue details sidebar, widens the sidebar if the detail texts are longer than it. It also adds the missing declarativeModel method to DeclarativeDataPlugin class.</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;">Printed the API key using qDebug(), got a string that looks like an API key.</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/plugins/declarative/DeclarativeDataPlugin.cpp <span style="color: grey">(73406b8)</span></li>

 <li>src/plugins/render/foursquare/FoursquarePlugin.h <span style="color: grey">(fc75f06)</span></li>

 <li>src/plugins/render/foursquare/FoursquarePlugin.cpp <span style="color: grey">(ae8210c)</span></li>

 <li>src/qt-components/marble-touch/activities/Explore.qml <span style="color: grey">(85d4344)</span></li>

</ul>

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




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








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