The jsdifflib's discuss from cemerick

Ignore whitespaces

Please make an option to ignore whitespaces and tabs

Broken link in readme

in README.asciidoc, the link

https://cemerick.github.com/jsdifflib/demo.html

that should redirect to a page where one can try the library, points to non existing page (404)

Feature add: Line numbers from git diff

If you have something like
@@ -130,18 +130,22 @@ class PostModel:

It would be great in view you can get real line numbers. Otherwise this is working great.

in the case of 'replace' if additional text or line found in base text should be in " red" (deleted) instead of "yellow" (replace) and if additional text found in the new text should be in green not in yellow

like in this in base text line 4 and 5 should in red and and in new text line 9 and 10 should be in green

WHY THE FOLLOWING CODEING COULDN'T PASS THE EXAM ON https://oj.bnuz.edu.cn/problemset/7/problem/592,BUT OK ON dev c++

WHAT IS WRONG? I KNOW THE PROGRAM IS FROM HERE, SO I MUST GET THE ANSWER FROM HERE. THIS KIND OF THING IS ALWAYS HAPPENED ON ME.

#include<stdio.h>
int main(){
int n;
scanf("%d",&n);
for(int i = 1;i <= n+1;i ++){
for(int j = 1;j <=2 * n+1;j ++){
if(j == n + i || j == n -i + 2){
printf("");
}
else{
printf(".");
}
}
printf("\n");
}
for(int i = n;i >= 1;i --){
for(int j = 2 * n+1;j >=1;j--){
if(j == n + i || j == n -i + 2){
printf("");
}
else{
printf(".");
}
}
printf("\n");
}
}

word level diff

how about to support word level diff ?
this is an example.

http://moniwiki.kldp.net/diff.html

Indentation problem

$$div.appendChild(diffview.buildView({
baseTextLines: base,
newTextLines: newtxt,
opcodes: opcodes,
// set the display titles for each resource
baseTextName: "Old version",
newTextName: "New version",
contextSize: 12,
viewType: 0
}));

baseTextLines and newTextLines has white spaces and well indented. But when it prints the value of base text and new text to the screen, they are not indented. What can be the problem?

Ignore diffs in comments

Some sugestion to do the SequenceMatcher ignore comments?

README incorrect - "latter" should be "former"

The Python Interoperability section says:

So, you have the choice of doing everything in Javascript on the browser, or falling back to server-side diff...

and then goes on to say:

Most of the time, we do the latter, simply because while jsdifflib is pretty fast all by itself, and is totally usable for diffing "normal" files....

This doesn't make sense. "latter" in that second quote should be "former".

0xc2132d05d31c914a87c6611c10748aeb04b58e8f

Feature - knockoutjs customBindingHandler

This is a simple binding handlers for knockout js.

    ko.bindingHandlers['jsDiffLib'] = {
            update: function (element, valueAccessor) {
                function getHtml(table){
                    var elems = [];
                    $(table).find("tbody>tr>td").each(function (item, elem) {
                        if(elem.className === "equal"){
                            elems.push("<span>" + elem.innerHTML + "</span>");
                        }
                        if(elem.className === "insert"){
                            elems.push("<ins>" + elem.innerHTML + "</ins>");
                        }
                        if(elem.className === "delete"){
                            elems.push("<del>" + elem.innerHTML + "</del>");
                        }
                    });
                    elems.splice(0, 1);
                    return elems.join("<br/>");
                }

                var options = valueAccessor();

                var base = difflib.stringAsLines("\n" + options.old()),
                    newtxt = difflib.stringAsLines("\n" + options.new()),
                    sm = new difflib.SequenceMatcher(base, newtxt),
                    opcodes = sm.get_opcodes();

                var currentTable = diffview.buildView({
                    baseTextLines: base,
                    newTextLines: newtxt,
                    opcodes: opcodes,
                    baseTextName: "Base Text",
                    newTextName: "New Text",
                    contextSize: null,
                    viewType: 1
                });
                var html = getHtml(currentTable);

                $(element).html(html);
            }
        }

Some information

@cemerick :
Hi congratulations on the module,
in Python there is: difflib.SequenceMatcher (None, str1, str2) .ratio ()
in the module you created there is a similar thing, does the same thing?

Example if I have:

murder on the orient express
murder on the express express 2017

The value of ratio, which value should you give according to your module?
Exceeds in this example the value of: 0.90?

P.Š.
Would it be possible to use it on React-native?

I thank you in advance.

It's okey

It's okey
That before source code is not valid
This one is okey and verify

Originally posted by @abbaskiko in #35 (comment)

does this works on nodejs or rhino ?

Highlight diff in an "updated line"

There seem to be 4 results: unchanged, added, removed, and updated.

For updated, it would be really cool if the library could highlight what exactly is different in an updated line like git's highlight-diff does.

question: fixing difview's width

I have text that contains some very long rows of data.

At the moment, the diffview table is expanding to the whole width of the data. Ideally, I would like the long lines of data to wrap - is this possible? Any tips?

Btw - I had looked into the CSS, but didn't get very far. I was hoping for an insider tip-off ;)

Many thanks!