Style fix for topAxis and added integration test

This commit is contained in:
Knut Sveidqvist
2021-04-25 10:41:59 +02:00
parent adab9819bd
commit f7750e5f04
3 changed files with 39 additions and 3 deletions

View File

@@ -258,4 +258,40 @@ describe('Gantt diagram', () => {
expect(svg).to.not.have.attr('style');
});
});
it('should render a gantt diagram with data labels at the top when topAxis is true', () => {
renderGraph(
`
gantt
dateFormat YYYY-MM-DD
axisFormat %d/%m
title Adding GANTT diagram to mermaid
excludes weekdays 2014-01-10
section A section
Completed task :done, des1, 2014-01-06,2014-01-08
Active task :active, des2, 2014-01-09, 3d
Future task : des3, after des2, 5d
Future task2 : des4, after des3, 5d
section Critical tasks
Completed task in the critical line :crit, done, 2014-01-06,24h
Implement parser and jison :crit, done, after des1, 2d
Create tests for parser :crit, active, 3d
Future task in critical line :crit, 5d
Create tests for renderer :2d
Add to mermaid :1d
section Documentation
Describe gantt syntax :active, a1, after des1, 3d
Add gantt diagram to demo page :after a1 , 20h
Add another diagram to demo page :doc1, after a1 , 48h
section Last section
Describe gantt syntax :after doc1, 3d
Add gantt diagram to demo page : 20h
Add another diagram to demo page : 48h
`,
{ gantt: { topAxis: true } }
);
});
});

View File

@@ -616,7 +616,7 @@ const config = {
*| --- | --- | --- | --- | --- |
*| topAxis | See notes | Boolean | 4 | True, False |
*
***Notes:**when this flag is set date labels will be added to the
***Notes:** when this flag is set date labels will be added to the
top of the chart
*
***Default value false**.

View File

@@ -377,8 +377,8 @@ export const draw = function(text, id) {
.style('text-anchor', 'middle')
.attr('fill', '#000')
.attr('stroke', 'none')
.attr('font-size', 10)
.attr('dy', '1em');
.attr('font-size', 10);
// .attr('dy', '1em');
}
}