Skip to content

Commit 4b13c4a

Browse files
committed
refactor!: transform StyleDefault into an object
`StyleDefault` was previously an enum but there is no need to enumerate the values. In addition, the values store both number and string which is generally considered confusing and a bad practice. Switching to a regular object make things more explicit and fix 23 SonarCloud code smells.
1 parent f076ee5 commit 4b13c4a

File tree

1 file changed

+35
-35
lines changed

1 file changed

+35
-35
lines changed

src/component/mxgraph/style/utils.ts

Lines changed: 35 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -25,45 +25,45 @@ import { BpmnStyleIdentifier } from './identifiers';
2525
* @category BPMN Theme
2626
* @experimental
2727
*/
28-
export enum StyleDefault {
29-
STROKE_WIDTH_THIN = 2,
30-
STROKE_WIDTH_THICK = 5,
31-
SHAPE_ACTIVITY_BOTTOM_MARGIN = 7,
32-
SHAPE_ACTIVITY_TOP_MARGIN = 7,
33-
SHAPE_ACTIVITY_LEFT_MARGIN = 7,
34-
SHAPE_ACTIVITY_FROM_CENTER_MARGIN = 7,
35-
SHAPE_ACTIVITY_MARKER_ICON_MARGIN = 5,
36-
SHAPE_ACTIVITY_MARKER_ICON_SIZE = 20,
37-
POOL_LABEL_SIZE = 30, // most of BPMN pool are ok when setting it to 30
38-
POOL_LABEL_FILL_COLOR = 'none',
39-
LANE_LABEL_SIZE = 30, // most of BPMN lane are ok when setting it to 30
40-
LANE_LABEL_FILL_COLOR = 'none',
41-
SUB_PROCESS_TRANSACTION_INNER_RECT_OFFSET = 4,
42-
SUB_PROCESS_TRANSACTION_INNER_RECT_ARC_SIZE = 6,
43-
TEXT_ANNOTATION_BORDER_LENGTH = 10,
44-
TEXT_ANNOTATION_FILL_COLOR = 'none',
45-
GROUP_FILL_COLOR = 'none',
28+
export const StyleDefault = {
29+
STROKE_WIDTH_THIN: 2,
30+
STROKE_WIDTH_THICK: 5,
31+
SHAPE_ACTIVITY_BOTTOM_MARGIN: 7,
32+
SHAPE_ACTIVITY_TOP_MARGIN: 7,
33+
SHAPE_ACTIVITY_LEFT_MARGIN: 7,
34+
SHAPE_ACTIVITY_FROM_CENTER_MARGIN: 7,
35+
SHAPE_ACTIVITY_MARKER_ICON_MARGIN: 5,
36+
SHAPE_ACTIVITY_MARKER_ICON_SIZE: 20,
37+
POOL_LABEL_SIZE: 30, // most of BPMN pool are ok when setting it to 30
38+
POOL_LABEL_FILL_COLOR: 'none',
39+
LANE_LABEL_SIZE: 30, // most of BPMN lane are ok when setting it to 30
40+
LANE_LABEL_FILL_COLOR: 'none',
41+
SUB_PROCESS_TRANSACTION_INNER_RECT_OFFSET: 4,
42+
SUB_PROCESS_TRANSACTION_INNER_RECT_ARC_SIZE: 6,
43+
TEXT_ANNOTATION_BORDER_LENGTH: 10,
44+
TEXT_ANNOTATION_FILL_COLOR: 'none',
45+
GROUP_FILL_COLOR: 'none',
4646
// General
47-
DEFAULT_FILL_COLOR = 'White',
48-
DEFAULT_STROKE_COLOR = 'Black',
49-
DEFAULT_FONT_FAMILY = 'Arial, Helvetica, sans-serif', // define our own to not depend on eventual mxGraph default change
50-
DEFAULT_FONT_SIZE = 11,
51-
DEFAULT_FONT_COLOR = 'Black',
52-
DEFAULT_MARGIN = 0,
47+
DEFAULT_FILL_COLOR: 'White',
48+
DEFAULT_STROKE_COLOR: 'Black',
49+
DEFAULT_FONT_FAMILY: 'Arial, Helvetica, sans-serif', // define our own to not depend on eventual mxGraph default change
50+
DEFAULT_FONT_SIZE: 11,
51+
DEFAULT_FONT_COLOR: 'Black',
52+
DEFAULT_MARGIN: 0,
5353
// Shape defaults
54-
SHAPE_ARC_SIZE = 20,
54+
SHAPE_ARC_SIZE: 20,
5555
// Overlay defaults
56-
DEFAULT_OVERLAY_FILL_COLOR = DEFAULT_FILL_COLOR,
57-
DEFAULT_OVERLAY_FILL_OPACITY = 100,
58-
DEFAULT_OVERLAY_STROKE_COLOR = DEFAULT_STROKE_COLOR,
59-
DEFAULT_OVERLAY_STROKE_WIDTH = 1,
60-
DEFAULT_OVERLAY_FONT_SIZE = DEFAULT_FONT_SIZE,
61-
DEFAULT_OVERLAY_FONT_COLOR = DEFAULT_FONT_COLOR,
56+
DEFAULT_OVERLAY_FILL_COLOR: 'White',
57+
DEFAULT_OVERLAY_FILL_OPACITY: 100,
58+
DEFAULT_OVERLAY_STROKE_COLOR: 'Black',
59+
DEFAULT_OVERLAY_STROKE_WIDTH: 1,
60+
DEFAULT_OVERLAY_FONT_SIZE: 11,
61+
DEFAULT_OVERLAY_FONT_COLOR: 'Black',
6262
// Edge
63-
SEQUENCE_FLOW_CONDITIONAL_FROM_ACTIVITY_MARKER_FILL_COLOR = 'White',
64-
MESSAGE_FLOW_MARKER_START_FILL_COLOR = 'White',
65-
MESSAGE_FLOW_MARKER_END_FILL_COLOR = 'White',
66-
}
63+
SEQUENCE_FLOW_CONDITIONAL_FROM_ACTIVITY_MARKER_FILL_COLOR: 'White',
64+
MESSAGE_FLOW_MARKER_START_FILL_COLOR: 'White',
65+
MESSAGE_FLOW_MARKER_END_FILL_COLOR: 'White',
66+
};
6767

6868
/**
6969
* Get the BPMN 'instantiate' information from the style.

0 commit comments

Comments
 (0)