Skip to content
This repository has been archived by the owner on Jun 25, 2020. It is now read-only.

Fix WebSequenceDiagram style bug:the style was not used #194

Merged
merged 1 commit into from
Jan 24, 2015

Conversation

laedit
Copy link
Member

@laedit laedit commented Jan 24, 2015

Now the default style is really 'default', like indicated on https://www.websequencediagrams.com/embedding.html.
And with "@@sequence style", 'style' will replace 'default'.

@laedit laedit changed the title The style was not used Fix WebSequenceDiagram style bug:the style was not used Jan 24, 2015
<script type=""text/javascript"" src=""http://www.websequencediagrams.com/service.js""></script>";

var markdown = transform.Transform(input);
Assert.Equal(expected.RemoveWhiteSpace(), markdown.RemoveWhiteSpace());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ApprovalTests is awesome for this sort of test. https:/approvals/ApprovalTests.Net

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the tip!

JakeGinnivan added a commit that referenced this pull request Jan 24, 2015
Fix WebSequenceDiagram style bug:the style was not used
@JakeGinnivan JakeGinnivan merged commit ae567bc into master Jan 24, 2015
@JakeGinnivan JakeGinnivan deleted the Fix-WebSequenceDiagram-style-bug branch January 24, 2015 11:10
laedit pushed a commit that referenced this pull request Jan 25, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants