@syncfusion/ej2-gantt 20.2.43 → 20.2.46

package/dist/es6/ej2-gantt.es2015.js

@@ -366,7 +366,7 @@ class DateProcessor {
             if (hour < this.parent.defaultStartTime) {
                 this.setTime(this.parent.defaultStartTime, cloneDate);
             }
-            else if (hour >= this.parent.defaultEndTime) {
+            else if (hour > this.parent.defaultEndTime) {
                 cloneDate.setDate(cloneDate.getDate() + 1);
                 this.setTime(this.parent.defaultStartTime, cloneDate);
             }
@@ -1853,7 +1853,10 @@ class TaskProcessor extends DateProcessor {
             this.setTime(this.parent.defaultEndTime, baselineEndDate);
         }
         if ((ganttProperties.baselineStartDate && baselineEndDate &&
-            (ganttProperties.baselineStartDate.getTime() > baselineEndDate.getTime())) || ganttProperties.isMilestone) {
+            (ganttProperties.baselineStartDate.getTime() > baselineEndDate.getTime())) ||
+            ((!isNullOrUndefined(ganttProperties.baselineStartDate) && !isNullOrUndefined(ganttProperties.startDate) && (ganttProperties.baselineStartDate.getTime() === ganttProperties.startDate.getTime()))
+                && (!isNullOrUndefined(baselineEndDate) && !isNullOrUndefined(ganttProperties.endDate) && (baselineEndDate.toLocaleDateString() === ganttProperties.endDate.toLocaleDateString())) &&
+                ganttProperties.isMilestone)) {
             baselineEndDate = ganttProperties.baselineStartDate;
         }
         this.parent.setRecordValue('baselineEndDate', this.checkBaselineEndDate(baselineEndDate, ganttProperties), ganttProperties, true);
@@ -4822,7 +4825,7 @@ class GanttChart {
      * @private
      */
     expandedGanttRow(args) {
-        if (isNullOrUndefined(args['gridRow']) && this.parent.enableVirtualization) {
+        if ((isNullOrUndefined(args['gridRow']) && this.parent.enableVirtualization) || isNullOrUndefined(args['chartRow'])) {
             return;
         }
         const record = getValue('data', args);
@@ -5067,7 +5070,7 @@ class GanttChart {
                 if (isTab) {
                     if (this.parent.editSettings.allowNextRowEdit) {
                         const rowData = this.parent.currentViewData[this.focusedRowIndex];
-                        const columnName = this.parent.ganttColumns[nextElement.getAttribute('aria-colindex')].field;
+                        const columnName = this.parent.ganttColumns[nextElement.getAttribute('data-colindex')].field;
                         if (rowData.hasChildRecords) {
                             if (columnName === this.parent.taskFields.endDate || columnName ===
                                 this.parent.taskFields.duration || columnName === this.parent.taskFields.dependency ||
@@ -5081,7 +5084,9 @@ class GanttChart {
                             }
                             else {
                                 this.parent.treeGrid.grid.notify('key-pressed', e);
-                                this.parent.treeGrid.editCell(this.focusedRowIndex, columnName); // eslint-disable-line
+                                if (isInEditedState) {
+                                    this.parent.treeGrid.editCell(this.focusedRowIndex, columnName); // eslint-disable-line
+                                }
                             }
                         }
                         else {
@@ -5301,6 +5306,18 @@ class GanttChart {
                     childElement = element;
                 }
             }
+            if (element.classList.contains('e-right-label-temp-container') || element.classList.contains('e-left-label-temp-container') || element.classList.contains('e-indicator-span')) {
+                if (focus === 'add') {
+                    element.setAttribute('tabIndex', '0');
+                    addClass([element], 'e-active-container');
+                    element.focus();
+                }
+                else {
+                    removeClass([element], 'e-active-container');
+                    element.setAttribute('tabIndex', '-1');
+                    element.blur();
+                }
+            }
             if (focus === 'add' && !isNullOrUndefined(childElement)) {
                 element.setAttribute('tabIndex', '0');
                 addClass([childElement], 'e-active-container');
@@ -6262,7 +6279,7 @@ class Timeline {
                 endDate.setHours(24, 0, 0, 0);
             }
         }
-        if (this.isZooming || this.parent.isLoad) {
+        if ((this.isZooming || this.parent.isLoad) && isNullOrUndefined(this.parent.projectEndDate)) {
             this.updateTimelineAfterZooming(endDate, false);
         }
         return endDate;
@@ -9320,7 +9337,10 @@ class ChartRows extends DateProcessor {
             }
             if (this.parent.renderBaseline && this.templateData.ganttProperties.baselineStartDate &&
                 this.templateData.ganttProperties.baselineEndDate) {
-                taskBaselineTemplateNode = this.getTaskBaselineNode();
+                taskBaselineTemplateNode = ((!isNullOrUndefined(this.templateData.ganttProperties.baselineStartDate) && !isNullOrUndefined(this.templateData.ganttProperties.startDate) && (this.templateData.ganttProperties.baselineStartDate.getTime() === this.templateData.ganttProperties.startDate.getTime()))
+                    && (!isNullOrUndefined(this.templateData.ganttProperties.baselineEndDate) && !isNullOrUndefined(this.templateData.ganttProperties.endDate) && (this.templateData.ganttProperties.baselineEndDate.toLocaleDateString() === this.templateData.ganttProperties.endDate.toLocaleDateString())) &&
+                    this.templateData.ganttProperties.isMilestone)
+                    ? this.getMilestoneBaselineNode() : this.getTaskBaselineNode();
             }
         }
         else if (this.templateData.ganttProperties.isMilestone) {
@@ -9330,7 +9350,10 @@ class ChartRows extends DateProcessor {
             }
             if (this.parent.renderBaseline && this.templateData.ganttProperties.baselineStartDate &&
                 this.templateData.ganttProperties.baselineEndDate) {
-                taskBaselineTemplateNode = this.getMilestoneBaselineNode();
+                taskBaselineTemplateNode = ((!isNullOrUndefined(this.templateData.ganttProperties.baselineStartDate) && !isNullOrUndefined(this.templateData.ganttProperties.startDate) && (this.templateData.ganttProperties.baselineStartDate.getTime() === this.templateData.ganttProperties.startDate.getTime()))
+                    && (!isNullOrUndefined(this.templateData.ganttProperties.baselineEndDate) && !isNullOrUndefined(this.templateData.ganttProperties.endDate) && (this.templateData.ganttProperties.baselineEndDate.toLocaleDateString() === this.templateData.ganttProperties.endDate.toLocaleDateString())) &&
+                    this.templateData.ganttProperties.isMilestone)
+                    ? this.getMilestoneBaselineNode() : this.getTaskBaselineNode();
             }
         }
         else {
@@ -9375,7 +9398,10 @@ class ChartRows extends DateProcessor {
             }
             if (this.parent.renderBaseline && this.templateData.ganttProperties.baselineStartDate &&
                 this.templateData.ganttProperties.baselineEndDate) {
-                taskBaselineTemplateNode = this.getTaskBaselineNode();
+                taskBaselineTemplateNode = ((!isNullOrUndefined(this.templateData.ganttProperties.baselineStartDate) && !isNullOrUndefined(this.templateData.ganttProperties.startDate) && (this.templateData.ganttProperties.baselineStartDate.getTime() === this.templateData.ganttProperties.startDate.getTime()))
+                    && (!isNullOrUndefined(this.templateData.ganttProperties.baselineEndDate) && !isNullOrUndefined(this.templateData.ganttProperties.endDate) && (this.templateData.ganttProperties.baselineEndDate.toLocaleDateString() === this.templateData.ganttProperties.endDate.toLocaleDateString())) &&
+                    this.templateData.ganttProperties.isMilestone)
+                    ? this.getMilestoneBaselineNode() : this.getTaskBaselineNode();
             }
         }
         if (!this.templateData.hasChildRecords) {
@@ -11509,7 +11535,9 @@ class Splitter$1 {
                         this.splitterObject.paneSettings[1].size = null;
                         this.splitterObject.paneSettings[1].size = this.getSpliterPositionInPercentage(this.splitterPreviousPositionChart);
                     }
-                    this.parent.timelineModule.updateTimelineAfterZooming(this.parent.timelineModule.timelineEndDate, true);
+                    if (isNullOrUndefined(this.parent.projectEndDate)) {
+                        this.parent.timelineModule.updateTimelineAfterZooming(this.parent.timelineModule.timelineEndDate, true);
+                    }
                     callBackPromise.resolve(splitterResizedArgs);
                 });
                 return callBackPromise;
@@ -12726,6 +12754,16 @@ let Gantt = class Gantt extends Component {
             this.renderChartVerticalLines();
         }
     }
+    removeCriticalPathStyles() {
+        const ganttChartElement = this.ganttChartModule.chartElement;
+        removeClass(ganttChartElement.querySelectorAll('.e-gantt-child-taskbar-inner-div'), criticalChildTaskBarInnerDiv);
+        removeClass(ganttChartElement.querySelectorAll('.e-gantt-child-progressbar-inner-div'), criticalChildProgressBarInnerDiv);
+        removeClass(ganttChartElement.querySelectorAll('.e-milestone-top'), criticalMilestoneTop);
+        removeClass(ganttChartElement.querySelectorAll('.e-milestone-bottom'), criticalMilestoneBottom);
+        removeClass(this.element.querySelectorAll('.e-line'), criticalConnectorLine);
+        removeClass(this.element.querySelectorAll('.e-connector-line-right-arrow'), criticalConnectorLineRightArrow);
+        removeClass(this.element.querySelectorAll('.e-connector-line-left-arrow'), criticalConnectorLineLeftArrow);
+    }
     wireEvents() {
         if (this.allowKeyboard) {
             this.keyboardModule = new KeyboardEvents(this.element, {
@@ -12888,139 +12926,140 @@ let Gantt = class Gantt extends Component {
      * @private
      */
     getZoomingLevels() {
+        const _WeekStartDay = this.timelineSettings.weekStartDay;
         const zoomingLevels = [
             {
                 topTier: { unit: 'Year', format: 'yyyy', count: 50 },
                 bottomTier: { unit: 'Year', format: 'yyyy', count: 10 }, timelineUnitSize: 99, level: 0,
-                timelineViewMode: 'Year', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Year', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Year', format: 'yyyy', count: 20 },
                 bottomTier: { unit: 'Year', format: 'yyyy', count: 5 }, timelineUnitSize: 99, level: 1,
-                timelineViewMode: 'Year', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Year', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Year', format: 'yyyy', count: 5 },
                 bottomTier: { unit: 'Year', format: 'yyyy', count: 1 }, timelineUnitSize: 99, level: 2,
-                timelineViewMode: 'Year', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Year', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Year', format: 'MMM, yy', count: 1 },
                 bottomTier: {
                     unit: 'Month', formatter: this.displayHalfValue, count: 6
                 }, timelineUnitSize: 66, level: 3,
-                timelineViewMode: 'Year', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Year', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Year', format: 'MMM, yy', count: 1 },
                 bottomTier: {
                     unit: 'Month', formatter: this.displayHalfValue, count: 6
                 }, timelineUnitSize: 99, level: 4,
-                timelineViewMode: 'Year', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Year', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Year', format: 'MMM, yy', count: 1 },
                 bottomTier: {
                     unit: 'Month', formatter: this.displayQuarterValue, count: 3
                 }, timelineUnitSize: 66, level: 5,
-                timelineViewMode: 'Year', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Year', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Year', format: 'yyyy', count: 1 },
                 bottomTier: {
                     unit: 'Month', formatter: this.displayQuarterValue, count: 3
                 }, timelineUnitSize: 99, level: 6,
-                timelineViewMode: 'Year', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Year', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Year', format: 'yyyy', count: 1 },
                 bottomTier: { unit: 'Month', format: 'MMM yyyy', count: 1 }, timelineUnitSize: 99, level: 7,
-                timelineViewMode: 'Year', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Year', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Month', format: 'MMM, yy', count: 1 },
                 bottomTier: { unit: 'Week', format: 'dd', count: 1 }, timelineUnitSize: 33, level: 8,
-                timelineViewMode: 'Month', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Month', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Month', format: 'MMM, yyyy', count: 1 },
                 bottomTier: { unit: 'Week', format: 'dd MMM', count: 1 }, timelineUnitSize: 66, level: 9,
-                timelineViewMode: 'Month', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Month', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Month', format: 'MMM, yyyy', count: 1 },
                 bottomTier: { unit: 'Week', format: 'dd MMM', count: 1 }, timelineUnitSize: 99, level: 10,
-                timelineViewMode: 'Month', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Month', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Week', format: 'MMM dd, yyyy', count: 1 },
                 bottomTier: { unit: 'Day', format: 'd', count: 1 }, timelineUnitSize: 33, level: 11,
-                timelineViewMode: 'Week', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Week', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Week', format: 'MMM dd, yyyy', count: 1 },
                 bottomTier: { unit: 'Day', format: 'd', count: 1 }, timelineUnitSize: 66, level: 12,
-                timelineViewMode: 'Week', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Week', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Week', format: 'MMM dd, yyyy', count: 1 },
                 bottomTier: { unit: 'Day', format: 'd', count: 1 }, timelineUnitSize: 99, level: 13,
-                timelineViewMode: 'Week', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Week', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Day', format: 'E dd yyyy', count: 1 },
                 bottomTier: { unit: 'Hour', format: 'hh a', count: 12 }, timelineUnitSize: 66, level: 14,
-                timelineViewMode: 'Day', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Day', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Day', format: 'E dd yyyy', count: 1 },
                 bottomTier: { unit: 'Hour', format: 'hh a', count: 12 }, timelineUnitSize: 99, level: 15,
-                timelineViewMode: 'Day', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Day', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Day', format: 'E dd yyyy', count: 1 },
                 bottomTier: { unit: 'Hour', format: 'hh a', count: 6 }, timelineUnitSize: 66, level: 16,
-                timelineViewMode: 'Day', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Day', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Day', format: 'E dd yyyy', count: 1 },
                 bottomTier: { unit: 'Hour', format: 'hh a', count: 6 }, timelineUnitSize: 99, level: 17,
-                timelineViewMode: 'Day', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Day', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Day', format: 'E dd yyyy', count: 1 },
                 bottomTier: { unit: 'Hour', format: 'hh a', count: 2 }, timelineUnitSize: 66, level: 18,
-                timelineViewMode: 'Day', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Day', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Day', format: 'E dd yyyy', count: 1 },
                 bottomTier: { unit: 'Hour', format: 'hh a', count: 2 }, timelineUnitSize: 99, level: 19,
-                timelineViewMode: 'Day', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Day', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Day', format: 'E dd yyyy', count: 1 },
                 bottomTier: { unit: 'Hour', format: 'hh a', count: 1 }, timelineUnitSize: 66, level: 20,
-                timelineViewMode: 'Day', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Day', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Day', format: 'E dd yyyy', count: 1 },
                 bottomTier: { unit: 'Hour', format: 'hh a', count: 1 }, timelineUnitSize: 99, level: 21,
-                timelineViewMode: 'Day', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Day', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Hour', format: 'ddd MMM, h a', count: 1 },
                 bottomTier: { unit: 'Minutes', format: 'mm', count: 30 }, timelineUnitSize: 66, level: 22,
-                timelineViewMode: 'Hour', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Hour', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Hour', format: 'ddd MMM, h a', count: 1 },
                 bottomTier: { unit: 'Minutes', format: 'mm', count: 15 }, timelineUnitSize: 66, level: 23,
-                timelineViewMode: 'Hour', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Hour', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             },
             {
                 topTier: { unit: 'Hour', format: 'ddd MMM, h a', count: 1 },
                 bottomTier: { unit: 'Minutes', format: 'mm', count: 1 }, timelineUnitSize: 66, level: 24,
-                timelineViewMode: 'Hour', weekStartDay: 0, updateTimescaleView: true, weekendBackground: null, showTooltip: true
+                timelineViewMode: 'Hour', weekStartDay: _WeekStartDay, updateTimescaleView: true, weekendBackground: null, showTooltip: true
             }
         ];
         return zoomingLevels;
@@ -13265,9 +13304,18 @@ let Gantt = class Gantt extends Component {
                     }
                     break;
                 case 'timezone':
-                case 'enableCriticalPath':
                     this.dataOperation.checkDataBinding(true);
                     break;
+                case 'enableCriticalPath':
+                    if (this.enableCriticalPath && this.criticalPathModule) {
+                        this.criticalPathModule.showCriticalPath(this.enableCriticalPath);
+                        let criticalModule = this.criticalPathModule;
+                        this.criticalPathModule.criticalConnectorLine(criticalModule.criticalPathCollection, criticalModule.detailPredecessorCollection, true, criticalModule.predecessorCollectionTaskIds);
+                    }
+                    else {
+                        this.removeCriticalPathStyles();
+                    }
+                    break;
                 case 'filterSettings':
                     this.treeGrid.filterSettings = getActualProperties(this.filterSettings);
                     this.treeGrid.dataBind();
@@ -15804,6 +15852,9 @@ class CellEdit {
             this.parent.editModule.endEditAction(args);
         }
         this.isCellEdit = false;
+        if (!isNullOrUndefined(this.parent.editModule.cellEditModule.editedColumn)) {
+            this.parent.editModule.cellEditModule.editedColumn = null;
+        }
         if (!isNullOrUndefined(this.parent.toolbarModule)) {
             this.parent.toolbarModule.refreshToolbarItems();
         }
@@ -23891,7 +23942,7 @@ class Edit$2 {
                     else {
                         if (this.parent.viewType === 'ProjectView') {
                             if ((rowPosition === 'Top' || rowPosition === 'Bottom') ||
-                                ((rowPosition === 'Above' || rowPosition === 'Below' || rowPosition === 'Child') && !args.data.parentItem)) {
+                                ((rowPosition === 'Above' || rowPosition === 'Below' || rowPosition === 'Child') || isNullOrUndefined(rowPosition) && !args.data.parentItem)) {
                                 if (args.data instanceof Array) {
                                     this.updateRealDataSource(args.data, rowPosition);
                                 }
@@ -27220,10 +27271,7 @@ class CriticalPath {
     /* eslint-disable-next-line */
     criticalConnectorLine(criticalPathIds, collection, condition, collectionTaskId) {
         const ganttChartElement = this.parent.ganttChartModule.chartElement;
-        removeClass(ganttChartElement.querySelectorAll('.e-gantt-child-taskbar-inner-div'), criticalChildTaskBarInnerDiv);
-        removeClass(ganttChartElement.querySelectorAll('.e-gantt-child-progressbar-inner-div'), criticalChildProgressBarInnerDiv);
-        removeClass(ganttChartElement.querySelectorAll('.e-milestone-top'), criticalMilestoneTop);
-        removeClass(ganttChartElement.querySelectorAll('.e-milestone-bottom'), criticalMilestoneBottom);
+        this.parent.removeCriticalPathStyles();
         for (let i = 0; i < criticalPathIds.length; i++) {
             let criticalData;
             if (this.parent.viewType === 'ProjectView') {
@@ -27258,9 +27306,6 @@ class CriticalPath {
                 }
             }
         }
-        removeClass(this.parent.element.querySelectorAll('.e-line'), criticalConnectorLine);
-        removeClass(this.parent.element.querySelectorAll('.e-connector-line-right-arrow'), criticalConnectorLineRightArrow);
-        removeClass(this.parent.element.querySelectorAll('.e-connector-line-left-arrow'), criticalConnectorLineLeftArrow);
         if (collection.length !== 0) {
             let index = 0;
             let currentdata;
@@ -28345,7 +28390,7 @@ class RowDD$1 {
                                 let currentData = this.parent.currentViewData.filter(function (e) {
                                     return e[uniqueTaskID] === ganttData[i][uniqueTaskID];
                                 })[0];
-                                if (currentData.index > droppedRecord.index) {
+                                if (currentData && currentData.index > droppedRecord.index) {
                                     endIndex = currentData.index;
                                     break;
                                 }
@@ -28357,7 +28402,7 @@ class RowDD$1 {
                                 let currentData = this.parent.currentViewData.filter(function (e) {
                                     return e[uniqueTaskID] === ganttData[i][uniqueTaskID];
                                 })[0];
-                                if (currentData.index > draggedRecord.index) {
+                                if (currentData && currentData.index > draggedRecord.index) {
                                     endIndex = currentData.index;
                                     break;
                                 }