concordancejs/concordance

Do you want to work on this issue?

You can request for a bounty in order to promote it!

Inline diffing should prioritize diffing whole words #29

sindresorhus posted onGitHub

Unless the words are very long. AFAIK that's how Git diffing works too.

We don't want this (current):

<img width="356" alt="screen shot 2017-06-27 at 20 46 09" src="https://user-images.githubusercontent.com/170270/27604484-af0a0eb6-5b79-11e7-95d6-12462cf509e6.png">


Do you have suggestions for when a word is too long?

I assume this should go up to the word boundary /\b/, not just whitespace?

posted by novemberborn over 7 years ago

Do you have suggestions for when a word is too long?

Something arbitrary. Maybe 30 characters? Wonder how the Git diff algorithm handles this.

I assume this should go up to the word boundary /\b/, not just whitespace?

Yes


Some relevant resources:

posted by sindresorhus over 7 years ago
posted by novemberborn about 6 years ago

I think we should disable word diffing until we can implement something better. The digging continues to be unusable. Iā€™m often using console log.

posted by sindresorhus about 6 years ago

@issuehunt has funded $80.00 to this issue.


posted by IssueHuntBot almost 6 years ago

Fund this Issue

$80.00
Funded
Only logged in users can fund an issue

Pull requests

Recent activities

issuehunt funded 80.00 for concordancejs/concordance# 29
almost 6 years ago