Skip to content

Add changed lines counter#1176

Closed
BernatBC wants to merge 12 commits intosourcegit-scm:developfrom
BernatBC:add-changed-lines-counter
Closed

Add changed lines counter#1176
BernatBC wants to merge 12 commits intosourcegit-scm:developfrom
BernatBC:add-changed-lines-counter

Conversation

@BernatBC
Copy link
Contributor

Added and removed lines are displayed on the commit details.
image

I also plan to add these stats to files. However, I'm not sure where to place them (2 and 3)?:

image

Another PR was created a while ago (#914), but I declined it.

@DarkDefender
Copy link

I'm not part of the development team, but I would prefer location 3.
This is because I think it will be too cluttered to have the changed lines in the file list view.

Of course it might be nice sometimes to see the number of lines that changed, but for that I think showing it in the diff view (location 3) is good enough.

That's how they do it here on github as well (display it in the same place as the file name in the diff view):
image

@BernatBC
Copy link
Contributor Author

I updated the PR to include the number of lines changed on files:
Screenshot From 2025-04-13 17-13-55

@love-linger
Copy link
Collaborator

I agree with @DarkDefender. For me, the count of changed lines is useless.

And in the INFORMATION view, the STAT part is easy to cause misunderstanding. I think most people will more easily understand it as "Added X file(s), Modified Y file(s), Deleted Z files"

@love-linger love-linger self-assigned this Apr 17, 2025
@love-linger love-linger added the not-planned It's not planned in the future label Apr 17, 2025
@love-linger love-linger force-pushed the develop branch 4 times, most recently from ed356cd to 2a43efd Compare April 17, 2025 07:43
@BernatBC BernatBC closed this Apr 21, 2025
@saierd
Copy link

saierd commented Oct 21, 2025

I would like this, both per file as well as in the commit overview.

And in the INFORMATION view, the STAT part is easy to cause misunderstanding. I think most people will more easily understand it as "Added X file(s), Modified Y file(s), Deleted Z files"

I disagree. If you would add the number of changes files as well, the proposed format is exactly what the widely known git show --stat and git show --shortstat displays.

This is how it looks in Sublime Merge, with a tooltip to explain the numbers:

Screenshot from 2025-10-21 10-28-38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-planned It's not planned in the future

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants