small change to delete key handling

app/actions.js

@@ -156,8 +156,7 @@ export function deleteCaption(captionId) {
   return { type: DELETE_CAPTION, captionId };
 }
 
-export function deleteSelection(event, selection) {
-  event.preventDefault();
+export function deleteSelection(selection) {
   return { type: DELETE_SELECTION, selection };
 }
 

app/components/Root.jsx

@@ -73,7 +73,10 @@ class Root extends Component {
       'resetZoom': () => dispatch(resetZoom()),
       'shiftDown': () => this.setState({ shiftKey: true }),
       'shiftUp': () => this.setState({ shiftKey: false }),
-      'delete': (event) => dispatch(deleteSelection(event, selection))
+      'delete': (event) => {
+        event.preventDefault();
+        dispatch(deleteSelection(selection));
+      }
     };
 
     let graphApi = {

app/components/__tests__/Edge-test.jsx

@@ -72,14 +72,4 @@ describe("Edge Component", () => {
     TestUtils.Simulate.click(select);
     expect(clickEdge.mock.calls[0][0]).toBe(data.id);
   });
-
-  it("should be removed when delete hotkey is pressed", () => {
-    let deleteEdge = jest.genMockFunction();
-    let edge = TestUtils.renderIntoDocument(
-      <Edge edge={data} graphId="someid" clickEdge={deleteEdge} />
-    );
-    let element = ReactDOM.findDOMNode(edge);
-    TestUtils.Simulate.keyDown(element, {key: "Delete", keyCode: 46, which: 46});
-    expect(deleteEdge.mock.calls[0]).toBeUndefined();
-  });
 });

app/components/__tests__/Node-test.jsx

@@ -81,14 +81,4 @@ describe("Node Component", () => {
 
     expect(moveNode.mock.calls.length).toBe(1);
   });
-
-  it("should be removed when delete hotkey is pressed", () => {
-    let deleteNode = jest.genMockFunction();
-    let wrapper = shallow(
-      <Node node={data} graph={{id: "someid"}} clickNode={deleteNode} />
-    );
-    let element = wrapper.find("g.node");
-    TestUtils.Simulate.keyDown(element, {key: "Delete", keyCode: 46, which: 46});
-    expect(deleteNode.mock.calls[0]).toBeUndefined();
-  });
 });

app/main.jsx

@@ -201,8 +201,8 @@ class Oligrapher {
     this.root.dispatchProps.dispatch(deselectAll());
   }
 
-  deleteSelection(event) {
-    this.root.dispatchProps.dispatch(deleteSelection(event, this.getSelection()));
+  deleteSelection() {
+    this.root.dispatchProps.dispatch(deleteSelection(this.getSelection()));
   }
 
   updateNode(nodeId, data) {