Skip to content

Conversation

@jfountain
Copy link

Unfortunately github seems to have discontinued the '?w=1' whitespace hiding feature. Noticed this about a week ago.

Removed reference to whitespace feature that is no longer supported.
Removed reference to whitespace feature that is no longer supported.
Removed reference to whitespace feature that is no longer supported.
Removed reference to whitespace feature that is no longer supported.
Removed reference to whitespace feature that is no longer supported.
@bobthecow
Copy link

It still works on commit page diffs (for example, see dkim1116@a016269?w=1 from #154)

@hkdobrev
Copy link
Contributor

hkdobrev commented Apr 6, 2016

I think they just didn't implement everything in the new PR commit pages. For example .diff and .patch links there are not working either, but working on the canonical commit URLs.

@jfountain
Copy link
Author

Interesting that it's not consistent. I'm going to contact github support directly on this. It may have been a regression bug on the new PR pages. Or maybe the feature is going to be removed on all diff pages. Either way I'll close this for now and re-open depending on feedback from github.

@jfountain jfountain closed this Apr 7, 2016
@hkdobrev hkdobrev mentioned this pull request Apr 7, 2016
@jfountain
Copy link
Author

I heard back from Github on this and they turned the feature back on for pull requests. All rejoice! Thanks @rsese

@jfountain jfountain deleted the remove-ignore-whitespace branch April 26, 2016 13:28
@tiimgreen
Copy link
Owner

Awesome, thanks!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants