Commit changes diff box does not account for code whitespaces/indentation #3983

Closed
opened 2025-11-02 05:32:47 -06:00 by GiteaMirror · 4 comments
Owner

Originally created by @cmarshall108 on GitHub (Sep 18, 2019).

The following screenshot is from gitea:

Screen Shot 2019-09-18 at 7 33 11 PM

The following screenshot is from github:

Screen Shot 2019-09-18 at 7 33 02 PM

Both of the repos are the same, and both of the commits above in the screenshots are the same commits. Gitea does not account for the code whitespaces like Github does, i found this a bit confusing and weird. I'm assuming this is not an intended function.

Originally created by @cmarshall108 on GitHub (Sep 18, 2019). The following screenshot is from gitea: <img width="1668" alt="Screen Shot 2019-09-18 at 7 33 11 PM" src="https://user-images.githubusercontent.com/22750553/65195148-fd627000-da4b-11e9-94d0-fbfef1509b68.png"> The following screenshot is from github: <img width="1679" alt="Screen Shot 2019-09-18 at 7 33 02 PM" src="https://user-images.githubusercontent.com/22750553/65195157-0a7f5f00-da4c-11e9-9f53-aa589716a715.png"> Both of the repos are the same, and both of the commits above in the screenshots are the same commits. Gitea does not account for the code whitespaces like Github does, i found this a bit confusing and weird. I'm assuming this is not an intended function.
GiteaMirror added the topic/uiissue/confirmedtype/bug labels 2025-11-02 05:32:48 -06:00
Author
Owner

@stale[bot] commented on GitHub (Nov 18, 2019):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale[bot] commented on GitHub (Nov 18, 2019): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.
Author
Owner

@zeripath commented on GitHub (Nov 18, 2019):

It's not the whitespace per se it's the word. Github is highlighting the words that change whereas we are highlighting only the minimal change.

I think GitHub is probably doing the correct thing even if it is not highlighting the minimal change

@zeripath commented on GitHub (Nov 18, 2019): It's not the whitespace per se it's the word. Github is highlighting the words that change whereas we are highlighting only the minimal change. I think GitHub is probably doing the correct thing even if it is not highlighting the minimal change
Author
Owner

@wxiaoguang commented on GitHub (Jul 5, 2022):

The diff seems correct on try.gitea.io (although it's different from github, but the spaces are correct)

2738b8e184

image

@wxiaoguang commented on GitHub (Jul 5, 2022): The diff seems correct on try.gitea.io (although it's different from github, but the spaces are correct) https://try.gitea.io/wxiaoguang/test/commit/2738b8e18402ed2793cb38e85d81f7da4d77d8d8 ![image](https://user-images.githubusercontent.com/2114189/177250410-71e74fec-55cb-4170-98b5-ff80f908a1f6.png)
Author
Owner

@wxiaoguang commented on GitHub (Jul 5, 2022):

ps: if you can not see whitespaces, make sure you are not affected by this : https://github.com/go-gitea/gitea/issues/19309

@wxiaoguang commented on GitHub (Jul 5, 2022): ps: if you can not see whitespaces, make sure you are not affected by this : https://github.com/go-gitea/gitea/issues/19309
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#3983