AssertException thrown if sequence diagram is not viewable

[wjam]

I currently have a reveal.js presentation with some sequence diagrams on some slides but not the first one. If the presentation is opened to a slide where none of the sequence diagrams are present, the following exception will be thrown and NONE of the sequence diagrams will be present.

uncaught exception: AssertException: x1,x2,y1,y2 must be numeric

Browser: Firefox 37.0.1

[bramp]

wjam@, would you be able to provide me a minimal working example of this error? (http://sscce.org/) Thanks.

[wjam]

I've just created a test case and it's as minimal as I can make it. I've also noticed that while Chrome doesn't display the same error, it does display all of the content squashed up.

It appears as though the combination of using Reveal.js with js-sequence-diagrams and JQuery plus having three sequence diagrams plus having three slides after the diagrams causes the bug!

If you replace JQuery with standard JavaScript then the problem disappears.
If you remove one of the slides after the diagrams then the problem disappears.
If you remove one of the diagrams then the problem disappears.

[bramp]

Thanks Will. I will investigate.
On 12 May 2015 1:26 am, "Will May" notifications@github.com wrote:

I've just created a test case
https://github.com/wjam/js-sequence-diagram-issue-99 and it's as
minimal as I can make it. I've also noticed that while Chrome doesn't
display the same error, it does display all of the content squashed up.

It appears as though the combination of using Reveal.js with
js-sequence-diagrams and JQuery plus having three sequence diagrams plus
having three slides after the diagrams causes the bug!

If you replace JQuery with standard JavaScript then the problem disappears.
If you remove one of the slides after the diagrams then the problem
disappears.
If you remove one of the diagrams then the problem disappears.

—
Reply to this email directly or view it on GitHub
#99 (comment)
.

[wjam]

Updated the test case to remove dependency on JQuery

[eueung]

I got the same issue when integrating this with remark.js

But this happens only when I use {theme: 'simple'}. Using {theme: 'hand'}, everything is ok. The issue might be related with theme initialization.

[jbe456]

I'm having the same issue when switching the theme to simple, whereas hand works fine