<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/123385/">https://git.reviewboard.kde.org/r/123385/</a>
     </td>
    </tr>
   </table>
   <br />




<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>

<div>Review request for Marble.</div>
<div>By Gábor Péterffy.</div>


<p style="color: grey;"><i>Updated ápr. 18, 2015, 3:12 du</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;">Files are put to the right place, and SessionElement has been separated from SessionManager.h.</pre>
  </td>
 </tr>
</table>







<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
marble
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This patch enables to have multiple sessions at the same time. It separates MarbleWidgets and some other objects with deep connection with it from ControlView, and manages them with SessionManager as SessionElements.
Adds the ability to start new sessions and later return to the previous ones.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">-This could be improved with adding an UI to change between sessions without closing any
-It could be used up to support high resolution printing with creating a session in the background
-This is necessary for the panorama viewer</p></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>src/apps/marble-ui/CMakeLists.txt <span style="color: grey">(a78467cfb9f9535b57cf2e816a85d55c63fa2a3f)</span></li>

 <li>src/apps/marble-ui/ControlView.h <span style="color: grey">(21d58cff305e8380cae1fb747936bafb4088ea5c)</span></li>

 <li>src/apps/marble-ui/ControlView.cpp <span style="color: grey">(a16aa28677ac92e9147dbfbf1be91e3f0c1951c8)</span></li>

 <li>src/apps/marble-ui/SessionElement.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/apps/marble-ui/SessionElement.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/apps/marble-ui/SessionManager.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/apps/marble-ui/SessionManager.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/lib/marble/CMakeLists.txt <span style="color: grey">(f448f8498fca34d06a8b41b3db4baf2286e06de1)</span></li>

 <li>src/lib/marble/FileViewWidget.cpp <span style="color: grey">(96a5a8ecd7c2d59adb1a11a9418beae9e496f7e2)</span></li>

 <li>src/lib/marble/MapViewWidget.cpp <span style="color: grey">(4d8f9f6a3f88300b4ee6ad5a9689ac838294b25a)</span></li>

</ul>

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






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



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