D29099: Make the Focus capture action include star detection
noreply at phabricator.kde.org
Wed Apr 22 16:05:49 BST 2020
TallFurryMan created this revision.
TallFurryMan added a reviewer: KStars.
Herald added a project: KDE Edu.
Herald added a subscriber: kde-edu.
TallFurryMan requested review of this revision.
This differential makes the Focus capture button state reflect exposure and detection.
Before that change, the capture button reverted to the enable state after the frame was received but the source extraction continued in the background.
The UI was therefore ready to accept user input, but at the same time unresponsive.
The differential adjusts the UI tests to wait for the capture button to re-enable, instead of inserting a suitable delay before waiting.
This differential also replaces an assertion generated when the tracking box is enabled in full-field detection, with an immediate disable of the tracking box.
This differential also rewrites the tooltip displayed by the detection method drop-down, providing more detailed information.
This differential also fixes the explicit assignment operator warnings.
UI tests through kstars_ui_tests. Jenkins builds for warnings.
improve__determine_star_detection_end (branched from master)
To: TallFurryMan, #kstars
Cc: kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the kde-edu