I encountered a problem when merging why some more commas

[djfly]

<h3>,   sfdfsdfsf,</h3>,<p>,    <br />,</p>,<hr />,,<p>,    <br />,</p>,<blockquote>,   fdsfdsfsdfdsf 45454545,</blockquote>

The Merge results will be a lot more commas?

[djfly]

@Xiphe Emergency help

[Xiphe]

Hey @djfly
Can you please provide the source documents that you're trying to merge?
Don't know how to reproduce this

[djfly]

in example/index.php

<script type="text/javascript">
            jQuery(document).ready(function($) {
                $('.Differences').phpdiffmerge({
                    left: left,
                    right: right,
                    pupupResult: true,
                    debug: true,
                    merged: function(a,b,c) {
                        alert(a); // add this look it
                        console.log('Merge completed.');
                    }
                });
            });
        </script>

[Xiphe]

yeah, i meant more the contents of left and right

[djfly]

left and right is your example default
you can just add alert(a); // add this look it in your example to reproduce it

[Xiphe]

aaah, sorry i accidentally deleted your first comment. Don't know how to restore it :(

[djfly]

Does not matter, are you reproduce it?

[Xiphe]

Sorry - I don't get it. The example uses a.txt and b.txt from the example folder there are no commas and no fdsfdsfsdfdsf 45454545, so i do not understand where that strings come from.

[djfly]

See Figure merge results yet? A lot more commas

[djfly]

Each line at end added ,

[Xiphe]

Ah, thats weird...
Which browser are you using?
Looks to me as if the way i open the popup does not go well with your browser.

[Xiphe]

Yeah, that's right. looks as if the browser uses , instead of \n...

[Xiphe]

the popup view originally was more a debugging tool. and i left it in because i thought that somebody might need it.

I'd recommend not using the build in popup thing and just use the merge result here.

[djfly]

in chrome and ie 11 both Have this problem, this is not browser problem
popup result is right
merge result is not right

[Xiphe]

i see, going to check that

[djfly]

Thank God, you finally understand.

[Xiphe]

:)

[Xiphe]

Got it! Since left and right are arrays of strings, the merge result is also an array. if you alert an array it will stringify that array using commas

alert(['foo', 'bar']); -> foo,bar

what you need to do is join the array with your favorite line break

alert(result.join("\n"));

going to make that more clear in the docs.

[djfly]

thanks, Finally solved.

[Xiphe]

you're welcome :)

[djfly]

Finally, my project is done！
visit:http://wegit.org/