Clean up code.

This commit is contained in:
Paulo Gustavo Veiga 2023-01-02 11:07:41 -08:00
parent fe8af2f7a5
commit 0a707d2860
2 changed files with 18 additions and 44 deletions

View File

@ -152,7 +152,7 @@ class StandaloneActionDispatcher extends ActionDispatcher {
const commandFunc = (topic: Topic, commandColor: string) => {
const result = topic.getFontColor();
topic.setFontColor(commandColor, true);
topic.setFontColor(commandColor);
return result;
};

View File

@ -211,14 +211,6 @@ abstract class Topic extends NodeGraph {
// Create inner box.
this._innerShape = this._buildShape(TopicConfig.INNER_RECT_ATTRIBUTES, this.getShapeType());
// Update bgcolor ...
const bgColor = this.getBackgroundColor();
this._setBackgroundColor(bgColor, false);
// Update border color ...
const brColor = this.getBorderColor();
this._setBorderColor(brColor, false);
// Define the pointer ...
if (!this.isCentralTopic() && !this.isReadOnly()) {
this._innerShape.setCursor('move');
@ -494,13 +486,11 @@ abstract class Topic extends NodeGraph {
return result;
}
setFontColor(value: string, updateModel?: boolean) {
const textShape = this.getTextShape();
textShape.setColor(value);
if (updateModel) {
const model = this.getModel();
model.setFontColor(value);
}
setFontColor(value: string) {
const model = this.getModel();
model.setFontColor(value);
this.redraw();
}
private _setText(text: string | null, updateModel?: boolean) {
@ -531,32 +521,26 @@ abstract class Topic extends NodeGraph {
}
setBackgroundColor(color: string | undefined): void {
this._setBackgroundColor(color, true);
const model = this.getModel();
model.setBackgroundColor(color);
this.redraw();
}
setConnectionStyle(type: LineType): void {
const model = this.getModel();
model.setConnectionStyle(type);
this.redraw(true);
}
setConnectionColor(value: string | undefined): void {
const model = this.getModel();
model.setConnectionColor(value);
this.redraw(true);
}
private _setBackgroundColor(color: string | undefined, updateModel: boolean) {
const innerShape = this.getInnerShape();
innerShape.setFill(color);
if (updateModel) {
const model = this.getModel();
model.setBackgroundColor(color);
}
}
getBackgroundColor(): string {
const model = this.getModel();
let result = model.getBackgroundColor();
@ -573,26 +557,12 @@ abstract class Topic extends NodeGraph {
}
setBorderColor(color: string | undefined): void {
this._setBorderColor(color, true);
const model = this.getModel();
model.setBorderColor(color);
this.redraw(true);
}
private _setBorderColor(color: string | undefined, updateModel: boolean): void {
if (updateModel) {
const model = this.getModel();
model.setBorderColor(color);
}
const bgColor = this.getBackgroundColor();
const connector = this.getShrinkConnector();
if (connector) {
connector.setColor(bgColor);
}
const innerShape = this.getInnerShape();
innerShape.setAttribute('strokeColor', color);
}
getBorderColor(): string {
const model = this.getModel();
let result = model.getBorderColor();
@ -1312,6 +1282,10 @@ abstract class Topic extends NodeGraph {
const bgColor = this.getBackgroundColor();
this.getInnerShape().setFill(bgColor);
// Update font ...
const fontColor = this.getFontColor();
textShape.setColor(fontColor);
// Force the repaint in case that the main topic color has changed.
if (this.getParent()) {
this._connector.setColor(borderColor);