<table><tr><td style="">TallFurryMan created this revision.<br />TallFurryMan added reviewers: mutlaqja, wreissenberger.<br />Restricted Application added a project: KDE Edu.<br />Restricted Application added a subscriber: kde-edu.<br />TallFurryMan requested review of this revision.
</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/D14665">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Add move buttons, refactor button state changes.</p>

<p>This change required refactoring to avoid mixing up button states. I<br />
believe this is too opaque for its own good. The scheduler UI states<br />
should be moved to their own state machine, QML perhaps?</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Create a multi-job scheduler list, with some jobs that will start asap, others that will start later on.<br />
Double-click to edit jobs, check that single-click will only select.<br />
Observe that the end-user may click to edit, but observe that fields would not match job line in that case, so learning curve is probably relatively flat for this change.<br />
Check add button changes to apply when editing, check other button states.<br />
Change fields, check validation actually change fields.<br />
[BUG] Note the bug with RA/DEC fields, which do not update the job properly. Another change which triggers a save is required to have it work properly.<br />
Click to select a job, move it up and down, check other jobs re-evaluate properly.<br />
Fill fields for a new job, click to select a job, add the new job, new job should be inserted before the selected job.<br />
[SHOULD WORK, NOT TESTED] Observe schedule inversion when two jobs that can start now change order.<br />
[SHOULD WORK, NOT TESTED] Double-click a job to edit it, change a field, select another job without validating, apply changes, proper job should update.<br />
Check button states and evaluation when loading a scheduler list from a file.<br />
[BUG] Note that sometimes no line is selected, but it's still possible to move jobs.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>improve__homogeneize_scheduler_capture_edition_modes (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14665">https://phabricator.kde.org/D14665</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kstars/ekos/scheduler/scheduler.cpp<br />
kstars/ekos/scheduler/scheduler.h<br />
kstars/ekos/scheduler/scheduler.ui<br />
kstars/ekos/scheduler/schedulerjob.h</div></div></div><br /><div><strong>To: </strong>TallFurryMan, mutlaqja, wreissenberger<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>