Fix bug: setEndPointHistory without setStartPointHistory

This commit is contained in:
Julia Radzhabova 2018-11-08 10:58:53 +03:00
parent 44e8976021
commit dd40a46fe3
8 changed files with 55 additions and 29 deletions

View file

@ -376,8 +376,10 @@ define([
onTransparencyChangeComplete: function(field, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = newValue;
this.api.setEndPointHistory();
this._transparencyApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._transparencyApplyFunc();
}
this._sendUndoPoint = true;
},
@ -515,8 +517,10 @@ define([
onGradientChangeComplete: function(slider, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = true;
this.api.setEndPointHistory();
this._gradientApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._gradientApplyFunc();
}
this._sendUndoPoint = true;
},

View file

@ -242,8 +242,10 @@ define([
onTransparencyChangeComplete: function(field, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = newValue;
this.api.setEndPointHistory();
this._transparencyApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._transparencyApplyFunc();
}
this._sendUndoPoint = true;
},
@ -381,8 +383,10 @@ define([
onGradientChangeComplete: function(slider, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = true;
this.api.setEndPointHistory();
this._gradientApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._gradientApplyFunc();
}
this._sendUndoPoint = true;
},

View file

@ -361,8 +361,10 @@ define([
onTransparencyChangeComplete: function(field, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = newValue;
this.api.setEndPointHistory();
this._transparencyApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._transparencyApplyFunc();
}
this._sendUndoPoint = true;
},
@ -496,8 +498,10 @@ define([
onGradientChangeComplete: function(slider, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = true;
this.api.setEndPointHistory();
this._gradientApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._gradientApplyFunc();
}
this._sendUndoPoint = true;
},

View file

@ -572,8 +572,10 @@ define([
onGradientChangeComplete: function(slider, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = true;
this.api.setEndPointHistory();
this._gradientApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._gradientApplyFunc();
}
this._sendUndoPoint = true;
},

View file

@ -363,8 +363,10 @@ define([
onTransparencyChangeComplete: function(field, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = newValue;
this.api.setEndPointHistory();
this._transparencyApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._transparencyApplyFunc();
}
this._sendUndoPoint = true;
},
@ -502,8 +504,10 @@ define([
onGradientChangeComplete: function(slider, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = true;
this.api.setEndPointHistory();
this._gradientApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._gradientApplyFunc();
}
this._sendUndoPoint = true;
},

View file

@ -3086,7 +3086,7 @@ define([
tab = {action: 'pivot', caption: me.textPivot};
$panel = me.getApplication().getController('PivotTable').createToolbarPanel();
if ($panel) {
me.toolbar.addTab(tab, $panel, 4);
me.toolbar.addTab(tab, $panel, 3);
me.toolbar.setVisible('pivot', true);
}
}
@ -3094,7 +3094,7 @@ define([
var tab = {action: 'review', caption: me.toolbar.textTabCollaboration};
var $panel = me.getApplication().getController('Common.Controllers.ReviewChanges').createToolbarPanel();
if ( $panel )
me.toolbar.addTab(tab, $panel, 5);
me.toolbar.addTab(tab, $panel, 4);
if ( config.isDesktopApp ) {
// hide 'print' and 'save' buttons group and next separator
@ -3111,7 +3111,7 @@ define([
tab = {action: 'protect', caption: me.toolbar.textTabProtect};
$panel = me.getApplication().getController('Common.Controllers.Protection').createToolbarPanel();
if ($panel)
me.toolbar.addTab(tab, $panel, 6);
me.toolbar.addTab(tab, $panel, 5);
}
}
}

View file

@ -372,8 +372,10 @@ define([
onTransparencyChangeComplete: function(field, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = newValue;
this.api.setEndPointHistory();
this._transparencyApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._transparencyApplyFunc();
}
this._sendUndoPoint = true;
},
@ -511,8 +513,10 @@ define([
onGradientChangeComplete: function(slider, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = true;
this.api.setEndPointHistory();
this._gradientApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._gradientApplyFunc();
}
this._sendUndoPoint = true;
},

View file

@ -364,8 +364,10 @@ define([
onTransparencyChangeComplete: function(field, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = newValue;
this.api.setEndPointHistory();
this._transparencyApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._transparencyApplyFunc();
}
this._sendUndoPoint = true;
},
@ -503,8 +505,10 @@ define([
onGradientChangeComplete: function(slider, newValue, oldValue){
clearInterval(this.updateslider);
this._sliderChanged = true;
this.api.setEndPointHistory();
this._gradientApplyFunc();
if (!this._sendUndoPoint) { // start point was added
this.api.setEndPointHistory();
this._gradientApplyFunc();
}
this._sendUndoPoint = true;
},