-
WHITFIELDRE email authored
* type hints and actions package Signed-off-by: Jose Borreguero <borreguero@gmail.com> * forward declarations to prevent import loops Signed-off-by: Jose Borreguero <borreguero@gmail.com> * forward declarations to prevent import loops Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Add buttons and combo box * Update function name * subscribing the presenter to the viewer Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Modify grid layout * Include correct import * Add actions.ui file * Tying the PeaksActionModel to the presenter Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Instantiating the peak actions Signed-off-by: Jose Borreguero <borreguero@gmail.com> * group box in ui file Signed-off-by: Jose Borreguero <borreguero@gmail.com> * append/remove PeaksWorkspace from the view Signed-off-by: Jose Borreguero <borreguero@gmail.com> * resolve circular imports Signed-off-by: Jose Borreguero <borreguero@gmail.com> * lazy connections between vier and presenter Signed-off-by: Jose Borreguero <borreguero@gmail.com> * getattr should've been hasattr Signed-off-by: Jose Borreguero <borreguero@gmail.com> * getattr should've been hasattr Signed-off-by: Jose Borreguero <borreguero@gmail.com> * route signals only when presenter is not None Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Hack to get everything showing, need to fix PeakAction size policy * Get remove_peaks working * Add a peak to the workspace * Ugly but have peak adding working * Add minimum size to PeakActionsView * Move coordinate transform to sliceview presenter * Move add_peak from PeakAction model to PeaksViewer model * Trigger ADS update when adding peaks from python * Check one button at a time * Disable pan and zoom while adding peaks * Add minimum width for actions widget * Force draw after workspace replaced * Deactivate peak adding if pan or zoom selected * Remove handle PeakActionsPresenter from PeaksViewerCollectionView Signed-off-by: Jose Borreguero <borreguero@gmail.com> * rename peaks_actions_view property Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Revert "rename peaks_actions_view property" This reverts commit 0b59766b. * Revert "Remove handle PeakActionsPresenter from PeaksViewerCollectionView" This reverts commit c3eeb15b . * PeakActionsPresenter not used in PeaksViewerCollectionView Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Remove actions presenter and view Signed-off-by: Jose Borreguero <borreguero@gmail.com> * move actions/view.py to peaksviewers/ Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Cleanup peak adding * Change 'button_press_event' connection * Fix button selections * Add tests for adding peaks in peaksviewer * Add test for sliceviewer.presenter/peak_add_delete * peak deletion deletes peak closest to input position Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Unify deleting/adding peaks Signed-off-by: Jose Borreguero <borreguero@gmail.com> * test delete peaks Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Don't need to include frame from sliceviewer, PeaksViewer knows this * Change SliceViewer.peak_add_delete to add_delete_peak to make it the same as PeaksViewerCollectionPresenter Also simplify PeaksViewerModel.delete_peak * Add peaksviewer/sliceviewer integration test for add/remove peaks * send AfterReplaceNotification after removal of a peak Signed-off-by: Jose Borreguero <borreguero@gmail.com> * Fix coordinate transform * Make PeaksViewerModel.delete_peak faster * Fix test_peak_add_delete_event * Fix workspace ordering with multiple peaks workspaces And cleanup some unused methods. * Add test for PeakActionView.set_peaksworkspace * Update docs * Change replace/rename handle to keep same position * Adjust test for changes to replace/rename handle * Set visibility specs * Update docs * Forgot to add new image * Fix two issue with peak overlay display, Should be checking all workspaces instead of just newly added workspaces. Deactivate peak adding/removing when peak overlay is closed Co-authored-by: Jose Borreguero <borreguero@gmail.com> Co-authored-by: Jenna Delozier <jennakate998@gmail.com> Co-authored-by: Jenna DeLozier <52164248+jenest@users.noreply.github.com>