Replace keypress with keydown to fix Measurement deletion from Lesion Table

This commit is contained in:
Erik Ziegler 2015-12-20 18:52:17 -05:00
parent 220ff065dc
commit 5eb8a717cb
3 changed files with 20 additions and 13 deletions

View File

@ -70,7 +70,7 @@
}
function keyDownCallback(e, eventData) {
var keyCode = eventData.keyCode;
var keyCode = eventData.which;
if (keyCode === keys.DELETE ||
(keyCode === keys.D && eventData.event.ctrlKey === true)) {

View File

@ -15,6 +15,12 @@ function doneCallback(measurementData, deleteTool) {
}
}
// Delete a lesion if Ctrl+D or DELETE is pressed while a lesion is selected
var keys = {
D: 68,
DELETE: 46
};
Template.lesionTableRow.events({
'dblclick .location': function() {
log.info('Double clicked on Lesion Location cell');
@ -26,16 +32,16 @@ Template.lesionTableRow.events({
changeLesionLocationCallback(measurementData, null, doneCallback);
},
'keypress .location': function(e) {
var keyCode = e.keyCode;
'keydown .location': function(e) {
var keyCode = e.which;
if (keyCode === keys.DELETE ||
(keyCode === keys.D && e.ctrlKey === true)) {
var currentMeasurement = Template.parentData(1);
var currentTimepointID = this.timepointID;
var currentMeasurement = this;
showConfirmDialog(function() {
log.info('Removing Lesion: ' + currentMeasurement._id);
clearMeasurementTimepointData(currentMeasurement._id, currentTimepointID);
Meteor.call("removeMeasurement", currentMeasurement._id, function(error, response) {
console.log('Removed!');
});
});
}
}

View File

@ -70,21 +70,22 @@ Template.lesionTableTimepointCell.events({
if (currentMeasurement.isTarget) {
showConfirmDialog(function() {
log.info('Confirm clicked!');
clearMeasurementTimepointData(currentMeasurement._id, currentTimepointID);
});
} else {
changeNonTargetResponse(measurementData, null, doneCallback);
}
},
'keypress .lesionTableTimepointCell': function(e) {
var keyCode = e.keyCode;
'keydown .lesionTableTimepointCell': function(e) {
var keyCode = e.which;
if (keyCode === keys.DELETE ||
(keyCode === keys.D && e.ctrlKey === true)) {
var currentMeasurement = Template.parentData(1);
log.info('Removing Lesion: ' + currentMeasurement._id);
// TODO = Add confirm dialog first!
clearMeasurementTimepointData(currentMeasurement._id, this.timepointID);
var currentTimepointID = this.timepointID;
showConfirmDialog(function() {
clearMeasurementTimepointData(currentMeasurement._id, currentTimepointID);
});
}
}
});