When entering a commit in path filtering mode, select the filtered path

[stefanhaller]

This can be useful if you want to do some operation on the filtered file, e.g. check it out using c, or enter the file to make a custom patch from it.

Addresses #4935.

[codacy-production]

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
Report missing for fb005841	✅ 100.00%

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (fb00584)	Report Missing	Report Missing	Report Missing
Head commit (7fe73c1)	58231	50652	86.98%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#4942)	87	87	100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct. ↩

[mikavilpas]

Works great with e.g. lazygit --filter pkg/gui/filetree/commit_file_tree_view_model.go

A minor finding: for absolute paths, --filter itself seems to work, but this new feature focuses on the first directory/file when opening the commit with enter.

Reproduction: lazygit --filter $(pwd)/pkg/gui/filetree/commit_file_tree_view_model.go

[stefanhaller]

Thanks for checking. I added a fixup (c2d83f9) to solve the issue with absolute paths, and also made it work on Windows when you call it with a path that contains backslashes.

I didn't notice these things because I never use the --filter option, I only ever filter from within lazygit using its ctrl-s menu, in which case neither problem occurs.

[mikavilpas]

Absolute paths also work great now! 👍🏻