Issue #290: rename dismissSelection to commitSelection

This commit is contained in:
Julian Descottes 2016-08-01 11:58:07 +02:00 committed by juliandescottes
parent fc7fc6d9b0
commit 69393127fa
4 changed files with 5 additions and 5 deletions

View File

@ -112,7 +112,7 @@
};
/**
* If the currently selected tool is a selection tool, call dismissSelection handler on
* If the currently selected tool is a selection tool, call commitSelection handler on
* the current tool instance.
*/
ns.SelectionManager.prototype.commit = function() {
@ -120,7 +120,7 @@
var isSelectionTool = tool instanceof pskl.tools.drawing.selection.BaseSelect;
if (isSelectionTool) {
var overlay = pskl.app.drawingController.overlayFrame;
tool.dismissSelection(overlay);
tool.commitSelection(overlay);
}
};

View File

@ -16,7 +16,7 @@
ns.AbstractDragSelect.prototype.onSelectStart_ = function (col, row, frame, overlay) {
if (this.hasSelection) {
this.hasSelection = false;
this.dismissSelection(overlay);
this.commitSelection(overlay);
} else {
this.hasSelection = true;
this.onDragSelectStart_(col, row);

View File

@ -113,7 +113,7 @@
/**
* Protected method, should be called when the selection is dismissed.
*/
ns.BaseSelect.prototype.dismissSelection = function (overlay) {
ns.BaseSelect.prototype.commitSelection = function (overlay) {
if (this.isMovingContent_) {
$.publish(Events.SELECTION_PASTE);
this.isMovingContent_ = false;

View File

@ -24,7 +24,7 @@
ns.ShapeSelect.prototype.onSelectStart_ = function (col, row, frame, overlay) {
if (this.hasSelection) {
this.hasSelection = false;
this.dismissSelection(overlay);
this.commitSelection(overlay);
} else {
this.hasSelection = true;
// From the pixel clicked, get shape using an algorithm similar to the paintbucket one: