concord.escape function img regex shouldn't be greedy

[andrewshell]

What I did:

I have an outline with html tags after an img tag. Ex: <img src="#" /><figcaption>Caption</figcaption>

What I expect to happen:

The < and > signs around figcaption should remain intact. Only the tags for img should be modified.

<img src="#" />&lt;figcaption&gt;Caption&lt;/figcaption&gt;

What happens:

The final > sign around /figcaption is converted to > while the end > of img remains intact.

<img src="#" /&gt;&lt;figcaption&gt;Caption&lt;/figcaption/>

The fix:

The regex on line 996 of concord.js https://github.com/scripting/concord/blob/master/concord.js#L996

Should change from: new RegExp("<"+tag+"((?!>).+)(/)?>","gi")

to:

new RegExp("<"+tag+"((?!>).+?)(/)?>","gi")

(see the ? after .+)