Merge pull request #35 from gen4sp/master

keyup instead of keydown
This commit is contained in:
Dylan Vorster
2017-03-29 21:43:43 +02:00
committed by GitHub

View File

@ -134,7 +134,7 @@ export class DiagramWidget extends React.Component<DiagramProps, DiagramState> {
componentWillUnmount(){ componentWillUnmount(){
this.props.diagramEngine.removeListener(this.state.diagramEngineListener); this.props.diagramEngine.removeListener(this.state.diagramEngineListener);
this.props.diagramEngine.setCanvas(null); this.props.diagramEngine.setCanvas(null);
window.removeEventListener('keydown',this.state.windowListener); window.removeEventListener('keyup',this.state.windowListener);
} }
componentWillUpdate(nextProps: DiagramProps){ componentWillUpdate(nextProps: DiagramProps){
@ -162,8 +162,7 @@ export class DiagramWidget extends React.Component<DiagramProps, DiagramState> {
//add a keyboard listener //add a keyboard listener
this.setState({ this.setState({
renderedNodes: true, renderedNodes: true,
windowListener: window.addEventListener('keydown',(event) => { windowListener: window.addEventListener('keyup',(event) => {
//delete all selected //delete all selected
if(event.keyCode === 46 || event.keyCode === 8){ if(event.keyCode === 46 || event.keyCode === 8){
_.forEach(this.props.diagramEngine.getDiagramModel().getSelectedItems(),(element) => { _.forEach(this.props.diagramEngine.getDiagramModel().getSelectedItems(),(element) => {
@ -175,7 +174,7 @@ export class DiagramWidget extends React.Component<DiagramProps, DiagramState> {
}); });
this.forceUpdate(); this.forceUpdate();
} }
}) },false)
}); });
window.focus(); window.focus();
} }