-
Notifications
You must be signed in to change notification settings - Fork 9
[Bug] Large Diff Button Display After Every Stage! #2347
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
There is no such option. |
That's unfortunate, I recently update Fork and this started happening. Might have to revert until this is fixed :( |
I think it worked this way all the time. I just tried an older version and the behavior is the same. |
Have you tried |
@DanPristupov : no, I experience the same 'large diff button pops up after every commit' only since the latest version: 2.52.0 I use Fork on 3 Mac's of which 2 are quite old, running OS X El Capitan, where I use Fork 2.16 Even if there are only 2 lines different, the 'large diff' button is shown. The problem is that this often happens, but not always, so I do not know the exact conditions to reproduce it. I recently switched to an Azure project, but I can try next week if the problem is reproducable in the Github project the Git project came from - if you are interested. |
@haraldblatand if lines are too long, then they are also 'too large to display' |
Sorry for the delay. Here is an example: Running on Mac OS 14.7.5, the file BaanvakkenAsInGit.tcd is the instance of the file Baanvakken.tcd as known in Git, BaanvakkenWithSomeDiffs.tcd is the instance of that file with only a few differences and no long lines. The files are maintained as Baanvakken.tcd in iTrain, a model railway program and are XML. Fork 2.52.0 shows the 'large diff' button see Fork2.52.0 MessageLargeDiffs.png and Fork2.52.0 TheseAreAllTheDiffs.png The two instances of Baanvakken.tcd are in Baanvakken.zip The 'large diff' button is often shown in 2.52.0 but not always in similar circumstances with say not big deal differences. The XML is in my opinion poor quality, but I don't know if some diff algorithm screws up. I also attached the zipped preferences of Fork. In my .gitattributes there is only one line: .tcd .xml com.DanPristupov.Fork.plist.zip |
Oh nice, how do I download |
Thank you! I can confirm 2.50.1 fixes my issue with the large diff. Specifically, I suspect the threshold or algorithm for detecting large diffs changed between 2.50.1 and 2.51.0. I highly prefer the user experience in 2.50.1. |
The limit has been existing for many years, however because of a bug it didn't work as expected. We fixed this recently :). I just checked, it seems that since that the performance of the text control got improved or the computers became much faster and now I don't see the freeze anymore. So, we'll increase the limit in the next update. @haraldblatand thank you for the example 👍. You helped me to find the problem quickly. |
Would it be possible to just make this a toggle-able setting? I personally find large diff blocking to be very distracting, even for large files. |
Uh oh!
There was an error while loading. Please reload this page.
For files that have large diffs, every time I stage one or more lines, the large diff prompt shows up! I already hit "load diff" once, am still working with the same file, it doesn't make sense to show it every time I stage 1 line.
How can I disable the large diff default? I want to load large diff by default.
The text was updated successfully, but these errors were encountered: