Hide #184200
Replies: 4 comments 3 replies
-
|
The “Hide whitespace changes” option still works, but it may look broken for large diffs or collapsed files. GitHub doesn’t always apply this setting when the file is too big or not fully loaded. Try expanding the file or adding ?w=1 to the PR URL to ignore whitespace changes. |
Beta Was this translation helpful? Give feedback.
-
|
The “Hide whitespace changes” feature is not actually broken, but it has limitations on GitHub’s diff viewer. In this case, the file is marked as “large diff – not rendered by default”, and for such files GitHub does not reliably apply whitespace filtering (even when w=1 is present). When the diff is partially rendered or collapsed, indentation-only changes can still appear as full diffs. A few things to note: GitHub only applies whitespace hiding correctly when the file is fully expanded and rendered Large diffs may bypass whitespace filtering due to performance constraints git diff -b locally is more reliable because it operates on the full file, not a partial render So the behavior you’re seeing is a GitHub UI limitation, not a change in how whitespace filtering works. Expanding the entire file or reviewing the diff locally is the most accurate workaround. |
Beta Was this translation helpful? Give feedback.
-
|
@Youjjwal I suspected the large diff was related, but could you clarify what you mean by "fully expanded and rendered" ?
Do you mean it only works for files that do render by default or are you referring to a workaround of some kind? |
Beta Was this translation helpful? Give feedback.
-
|
🕒 Discussion Activity Reminder 🕒 This Discussion has been labeled as dormant by an automated system for having no activity in the last 60 days. Please consider one the following actions: 1️⃣ Close as Out of Date: If the topic is no longer relevant, close the Discussion as 2️⃣ Provide More Information: Share additional details or context — or let the community know if you've found a solution on your own. 3️⃣ Mark a Reply as Answer: If your question has been answered by a reply, mark the most helpful reply as the solution. Note: This dormant notification will only apply to Discussions with the Thank you for helping bring this Discussion to a resolution! 💬 |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Select Topic Area
Question
Body
I noticed today that the "hide whitespace" option is not working any more??

Note how the function signatures are completely unchanged, other than indentation (verified in my editor).
Showing the diff with
git diff -bgives a very different result, so I'm not so sure that's a helpful comparison:Details
The pull request in question is https://github.com/apache/arrow-rs/pull/9144/changes?w=1 (note the
w=1).The file in question is a "large diff.. not rendered by default", not sure if that's related?
Beta Was this translation helpful? Give feedback.
All reactions