<table><tr><td style="">TallFurryMan requested changes to this revision.<br />TallFurryMan added a comment.<br />This revision now requires changes to proceed.
</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/D19648">View Revision</a></tr></table><br /><div><div><p><strong>Capture module.</strong><br />
Functionality tested OK.<br />
This is a change of UI behavior: now clicking once on a job does sync the left fields with the properties of the selected job, as mentioned in the changeset.<br />
However, it makes the end-user think the job can be modified in-place, while there is actually no effect. Counter-however, the change makes adding a new instance of an existing job easier.<br />
I will say the change makes the UI slightly better in terms of usability, but the ergonomy is still flawed.<br />
I observe that arrow buttons are always enabled even if the selected row is first or last while scheduler has this proper (but not always), but this is completely minor. The code doing that should probably be refactored somewhere.</p>

<p><strong>Scheduler module.</strong><br />
OK for the typo fix, obviously.<br />
Got an assertion abort related to null lead time, that is fixed by <a href="https://phabricator.kde.org/D19393" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19393</a>.<br />
Functionality is OK, it's cool to press the mouse button and move through jobs with their properties immediately displayed.<br />
However, it's possible to change the order of the schedule while the Scheduler is running.<br />
As a result, the schedule is reevaluated immediately, and if the job that is running is moved, states are reset and Scheduler is lost.<br />
As an example, the job that was slewing becomes idle and the Scheduler remains stuck in that situation.<br />
I'll double-check what the current unpatched situation is.</p></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/D19648">https://phabricator.kde.org/D19648</a></div></div><br /><div><strong>To: </strong>wreissenberger, mutlaqja, TallFurryMan<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>