HG_REPOSITORIES/LPP/SciQLOP_Repos/SciQLop Commit - r1082:eff9e479c5d5

Add action to remove the selected zone with the "del" buttons

trabillard -

r1082:eff9e479c5d5

parent child

Context file:

r1082:eff9e479c5d5

gui/include/Actions/SelectionZoneAction.h +5 0

                 /// Sets the function which determine if the action should be enabled or disabled
                 void setEnableFunction(EnableFunction fun);
+                /// Sets the shortcut displayed by the action.
+                /// Note: The shortcut is only displayed and not active because it is not permanently stored
+                void setDisplayedShortcut(const QKeySequence &shortcut);
+                QKeySequence displayedShortcut() const;
                 /// The name of the action
                 QString name() const noexcept;

gui/src/Actions/SelectionZoneAction.cpp +11 0

                 }
                 QString m_Name;
+                QKeySequence m_DisplayedShortcut;
                 SelectionZoneAction::ExecuteFunction m_Fun;
                 SelectionZoneAction::EnableFunction m_EnableFun = [](auto zones) { return true; };
             };
                 impl->m_EnableFun = std::move(fun);
             }
+            void SelectionZoneAction::setDisplayedShortcut(const QKeySequence &shortcut)
+            {
+                impl->m_DisplayedShortcut = shortcut;
+            }
+            QKeySequence SelectionZoneAction::displayedShortcut() const
+            {
+                return impl->m_DisplayedShortcut;
+            }
             QString SelectionZoneAction::name() const noexcept
             {
                 return impl->m_Name;

gui/src/Visualization/VisualizationActionManager.cpp +3 -1

             {
                 auto &actionController = sqpApp->actionsGuiController();
-                actionController.addSectionZoneAction("Remove Selected Zone(s)", [](auto zones) {
+                auto removeZonesAction
+                    = actionController.addSectionZoneAction("Remove Selected Zone(s)", [](auto zones) {
                           for (auto selectionZone : zones) {
                               if (auto graph = selectionZone->parentGraphWidget()) {
                                   graph->removeSelectionZone(selectionZone);
                               }
                           }
                       });
+                removeZonesAction->setDisplayedShortcut(QKeySequence::Delete);
                 auto alignEnableFuntion = [](auto items) { return items.count() > 1; };

gui/src/Visualization/VisualizationGraphWidget.cpp +8 -1

             void VisualizationGraphWidget::removeSelectionZone(VisualizationSelectionZoneItem *selectionZone)
             {
+                parentVisualizationWidget()->selectionZoneManager().setSelected(selectionZone, false);
+                if (impl->m_HoveredZone == selectionZone) {
+                    impl->m_HoveredZone = nullptr;
+                    setCursor(Qt::ArrowCursor);
+                }
                 impl->m_SelectionZones.removeAll(selectionZone);
                 plot().removeItem(selectionZone);
                 plot().replot(QCustomPlot::rpQueuedReplot);
-                parentVisualizationWidget()->selectionZoneManager().setSelected(selectionZone, false);
             }
             void VisualizationGraphWidget::undoZoom()
                     for (auto zoneAction : zoneActions) {
                         auto action = graphMenu.addAction(zoneAction->name());
                         action->setEnabled(zoneAction->isEnabled(selectedItems));
+                        action->setShortcut(zoneAction->displayedShortcut());
                         QObject::connect(action, &QAction::triggered,
                                          [zoneAction, selectedItems]() { zoneAction->execute(selectedItems); });
                     }

gui/src/Visualization/VisualizationSelectionZoneManager.cpp +1 -1

             {
                 if (value != item->selected()) {
                     item->setSelected(value);
-                    item->parentPlot()->replot();
+                    item->parentPlot()->replot(QCustomPlot::rpQueuedReplot);
                 }
                 if (!value && impl->m_SelectedItems.contains(item)) {

gui/src/Visualization/VisualizationWidget.cpp +14 0

             #include "Visualization/IVisualizationWidgetVisitor.h"
             #include "Visualization/VisualizationActionManager.h"
             #include "Visualization/VisualizationGraphWidget.h"
+            #include "Visualization/VisualizationSelectionZoneItem.h"
             #include "Visualization/VisualizationSelectionZoneManager.h"
             #include "Visualization/VisualizationTabWidget.h"
             #include "Visualization/VisualizationZoneWidget.h"
                 sqpApp->dragDropGuiController().addDragDropTabBar(ui->tabWidget->tabBar());
+                // Actions
                 impl->m_ActionManager.installSelectionZoneActions();
+                auto removeZoneAction = new QAction("Remove selected zone(s)");
+                removeZoneAction->setShortcut(QKeySequence::Delete);
+                connect(removeZoneAction, &QAction::triggered, [this]() {
+                    auto selection = impl->m_ZoneSelectionManager->selectedItems();
+                    for (auto selectionZone : selection) {
+                        if (auto graph = selectionZone->parentGraphWidget()) {
+                            graph->removeSelectionZone(selectionZone);
+                        }
+                    }
+                });
+                addAction(removeZoneAction);
                 // Adds default tab
                 addTabView();
             }

General Comments 0

You need to be logged in to leave comments. Login now

No TODOs yet

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages