<table><tr><td style="">mutlaqja added a reviewer: mutlaqja.<br />mutlaqja added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D14813">View Revision</a></tr></table><br /><div><div><p>Wow this is great and it's pretty much feature-complete! A few comments:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">We probably need to include this now in the Tools menu.</li>
<li class="remarkup-list-item">KStars XPlanet Viewer, while technically correct, is not a user-friendly name. We should probably use something like "Solar System Simulator" or something like that. Something obvious to new users.</li>
<li class="remarkup-list-item">Mouse wheel should probably control FOV.</li>
<li class="remarkup-list-item">For FOV --> None vs KStars is not clear. First of all, it's not really "None", there is FOV obviously so a better term should be used here. Default? 100%? Close.. I don't know. Also using the program name is also weird "KStars", we need another name for that. "Current" or whatever. This includes "Use KStars's FOV' in options as well.</li>
<li class="remarkup-list-item">Reset buttons should probably align (on 2nd and 3rd options).</li>
<li class="remarkup-list-item">The timeshift setting is great, but maybe we can add custom date/time widget along with "Play" and "Pause" buttons and a unit combo (secs/min/hours/months/years) so every second of KStars time translate to XPlanet simulation time? Maybe I want to view some event at some time in the future or past.</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14813">https://phabricator.kde.org/D14813</a></div></div><br /><div><strong>To: </strong>lancaster, mutlaqja<br /><strong>Cc: </strong>mutlaqja, kde-edu, narvaez, apol<br /></div>