Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove ignore whitespace #155

Closed
wants to merge 9 commits into from
Closed

Remove ignore whitespace #155

wants to merge 9 commits into from

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