Skip to content

Commit e2e5101

Browse files
Handle more override edge animate
1 parent c063b92 commit e2e5101

File tree

2 files changed

+24
-2
lines changed

2 files changed

+24
-2
lines changed

packages/mermaid/src/diagrams/flowchart/flowDb.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -94,10 +94,10 @@ export const addVertex = function (
9494
const edge = edges.find((e) => e.id === id);
9595
if (edge) {
9696
const edgeDoc = doc as EdgeMetaData;
97-
if (edgeDoc?.animate) {
97+
if (edgeDoc?.animate !== undefined) {
9898
edge.animate = edgeDoc.animate;
9999
}
100-
if (edgeDoc?.animation) {
100+
if (edgeDoc?.animation !== undefined) {
101101
edge.animation = edgeDoc.animation;
102102
}
103103
return;

packages/mermaid/src/diagrams/flowchart/parser/flow-node-data.spec.js

+22
Original file line numberDiff line numberDiff line change
@@ -378,6 +378,28 @@ describe('when parsing directions', function () {
378378
expect(data4Layout.edges[5].id).toEqual('L_A1_D1_0');
379379
});
380380

381+
it('should handle overriding edge animate again', function () {
382+
const res = flow.parser.parse(`flowchart TD
383+
A e1@--> B
384+
C e2@--> D
385+
E e3@--> F
386+
e1@{ animate: true }
387+
e2@{ animate: false }
388+
e3@{ animate: true }
389+
e3@{ animate: false }
390+
`);
391+
392+
const data4Layout = flow.parser.yy.getData();
393+
expect(data4Layout.nodes.length).toBe(6);
394+
expect(data4Layout.edges.length).toBe(3);
395+
expect(data4Layout.edges[0].id).toEqual('e1');
396+
expect(data4Layout.edges[0].animate).toEqual(true);
397+
expect(data4Layout.edges[1].id).toEqual('e2');
398+
expect(data4Layout.edges[1].animate).toEqual(false);
399+
expect(data4Layout.edges[2].id).toEqual('e3');
400+
expect(data4Layout.edges[2].animate).toEqual(false);
401+
});
402+
381403
it.skip('should be possible to use @ syntax to add labels with trail spaces', function () {
382404
const res = flow.parser.parse(
383405
`flowchart TB

0 commit comments

Comments
 (0)