fix: Merge pull request #103 from Hummer12007/del-shortcut

Added a shortcut to delete selected regions

Author: seveibar

src/Annotator/reducers/general-reducer.js

@@ -745,6 +745,13 @@ export default (state: MainLayoutState, action: Action) => {
         (activeImage.regions || []).filter((r) => r.id !== action.region.id)
       )
     }
+    case "DELETE_SELECTED_REGION": {
+      return setIn(
+        state,
+        [...pathToActiveImage, "regions"],
+        (activeImage.regions || []).filter((r) => !r.highlighted)
+      )
+    }
     case "HEADER_BUTTON_CLICKED": {
       const buttonName = action.buttonName.toLowerCase()
       switch (buttonName) {

src/MainLayout/types.js

@@ -148,6 +148,7 @@ export type Action =
   | {| type: "OPEN_REGION_EDITOR", region: Region |}
   | {| type: "CLOSE_REGION_EDITOR", region: Region |}
   | {| type: "DELETE_REGION", region: Region |}
+  | {| type: "DELETE_SELECTED_REGION" |}
   | {| type: "HEADER_BUTTON_CLICKED", buttonName: string |}
   | {| type: "SELECT_TOOL", selectedTool: ToolEnum |}
   | {| type: "CANCEL" |}

src/ShortcutsManager/index.js

@@ -53,6 +53,11 @@ export const defaultHotkeys = [
     id: "exit_sample",
     description: "Exit sample without saving",
   },
+  {
+    id: "delete_region",
+    description: "Delete selected region",
+    binding: "Delete",
+  },
 ]
 export const defaultKeyMap = {}
 for (const { id, binding } of defaultHotkeys) defaultKeyMap[id] = binding
@@ -120,6 +125,11 @@ export const useDispatchHotkeyHandlers = ({ dispatch }) => {
           buttonName: "Save",
         })
       },
+      delete_region: () => {
+        dispatch({
+          type: "DELETE_SELECTED_REGION",
+        })
+      },
       // TODO
       // exit_sample: () => {
       //   dispatch({