+2

File Log: filter files by the file

Clément Moyroud 6 months ago updated 6 months ago 5

When opening the file-specific log view and selecting two commits to show the diff, all the files modified between these two commits are shown. Since the intent is to look at the initially selected file, it would be much better if the filename was set by default in the filter, or maybe have an option to only show the logged file which would work across file renames.

Log window

By default, for a file log, the file is preselected in the Files view. Why hide all the other files?

I had not noticed the file highlight, maybe it's too subtle? :D Even though, with 35k files in the repository, the list changes and the selection moves around, it's kinda confusing. Note that in our case, a given user only looks at maybe a dozen files out of all the ones in the repository.

Do you change all 35k files in the commit or do you have enabled to show even the unchanged ones?

I tried this myself now and -- to my own surprise -- once you change the commit selection in the Graph view, the logged file will be pre-selected *again*. This is even the case if the previously selected file is also present in the new commit. It's also the case when changing the selection from a single commit to two commits to diff those commits. Furthermore, the selection is revealed. This is contrary to the repository root Log, which will try to preserve the selected file. Either way, can you please give it a try yourself and confirm that the logged file is reliably selected?

Yes, that's what I mentioned in my second comment, the file is selected so it's better than I initially thought, but the selection moves around and it's a flat list of files, so it's cumbersome.

I don't have it set to show unchanged files, but if a file is modified every few weeks and I want to see changes across a couple of months, I'm going to have a very long list of all the other files that have been modified in the meantime. 99% of these files are irrelevant to me.

It's not that it's broken, it's that it could be better :D