IE Support issue

[tylerlong]

Online demo: http://mdp.tylingsoft.com/ Please scroll to the bottom and you will see the flowchart and sequence diagram.

Tested on Windows 8.1 64 bit, IE 11. Here is a screenshot:

Yes I found this issue #99 I think I might be able to fix the flowchart issue.

But there is also issue with the sequence diagram. You can see that the two vertical lines are too long.

[knsv]

Thanks, will take a look!

[tylerlong]

I confirm that flowchart issue could be solved by:

var mermaid_config = {
  htmlLabels: false
};

[tylerlong]

I have found the root cause to the sequence diagram vertical line too long issue. It is caused by this line:

https://github.com/knsv/mermaid/blob/a38a156d3b26c21b715dbe9a9f47e3ff27edb9e4/src/diagrams/sequenceDiagram/svgDraw.js#L78

.attr("y2", 2000)

2000 is a hard coded large value.

[tylerlong]

A workaround for the sequence diagram issue is:

$('line[y2="2000"]').each(function(){
  $(this).attr('y2', $(this).closest('svg').attr('height') - 10);
});

[knsv]

Thanks!. Did you manage to fix the styling for the flowcharts? If so I can add that to the default styling mermaid.css

[tylerlong]

No, I didn't change the style of flowcharts. I fixed it by:

var mermaid_config = {
  htmlLabels: false
};

[knsv]

@tylerlong do you think I can close this one? I am going through the issues and compiling them to a roadmap.

[tylerlong]

Yes, you can close it. But please don't ignore the following content:

I have found the root cause to the sequence diagram vertical line too long issue. It is caused by this line:

https://github.com/knsv/mermaid/blob/a38a156d3b26c21b715dbe9a9f47e3ff27edb9e4/src/diagrams/sequenceDiagram/svgDraw.js#L78

.attr("y2", 2000)

2000 is a hard coded large value.

Since I've already found the root cause, it should be easy to fix.

[knsv]

OK, closing. The y2 200o is no longer there. It is initially set to to 200 but updated after rendering.

[madhu72]

Hi even added htmlLabels:false, I am not seeing labels in IE 11

[madhu72]