Skip to content
This repository was archived by the owner on Oct 5, 2021. It is now read-only.

Commit 4d08d29

Browse files
Merge pull request #501 from kaviththiranga/master
Fix if/else delete
2 parents 4fead06 + 5abc851 commit 4d08d29

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

modules/web/js/ballerina/views/if-else-statement-view.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -246,6 +246,8 @@ define(['require', 'lodash', 'log', 'property_pane_utils', './ballerina-statemen
246246
childrenView.stopListening();
247247
});
248248
d3.select("#_" +this._model.id).remove();
249+
this.getDiagramRenderingContext().getViewOfModel(this._model.getParent()).getStatementContainer()
250+
.removeInnerDropZone(this._model);
249251
this.getBoundingBox().w(0).h(0);
250252
};
251253

modules/web/js/ballerina/views/statement-container.js

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -454,10 +454,14 @@ define(['lodash', 'jquery', 'd3', 'log', 'd3utils', './point', './ballerina-view
454454
this._managedStatements.splice(childStatementIndex, 1);
455455

456456
if (this._widestStatementView === childStatementView) {
457-
// we have deleted the widest statement.
458-
this._widestStatementView = computeWidestStatementView(this._managedStatements,
459-
this.diagramRenderingContext);
460-
this._updateContainerWidth(this._widestStatementView.getBoundingBox().w());
457+
if(!_.isEmpty(this._managedStatements)) {
458+
// we have deleted the widest statement.
459+
this._widestStatementView = computeWidestStatementView(this._managedStatements,
460+
this.diagramRenderingContext);
461+
this._updateContainerWidth(this._widestStatementView.getBoundingBox().w());
462+
} else {
463+
this._widestStatementView = undefined;
464+
}
461465
}
462466
};
463467

0 commit comments

Comments
 (0)