I hoped they would fix line wrapping - which should be a relatively simply task - by now.<p>This is a very needed feature for projects that work a lot with text files (such as markdown) that do not automatically break on set widths (e.g. [1])<p>I've already contacted Github support with this request months ago, but no one's stepped up with a fix. Maybe this can get escalated somehow? Heck, I'd even submit a pull request if it was possible for github/github.<p>[1] - <a href="https://github.com/cryptoparty/handbook/pull/72/files" rel="nofollow">https://github.com/cryptoparty/handbook/pull/72/files</a>
Since we're touching code reviews, I'll plug my userscript here. It's useful for me for code reviews of big commits: you can fold the files, mark them as reviewed etc. and all of this is persisted in the local storage. You can install it from [1] if you know userscripts or read [2] otherwise.<p>[1] <a href="http://userscripts.org/scripts/show/153049" rel="nofollow">http://userscripts.org/scripts/show/153049</a><p>[2] <a href="https://github.com/jakub-g/gh-code-review-assistant" rel="nofollow">https://github.com/jakub-g/gh-code-review-assistant</a>
> and the feature is available anywhere we render diffs<p>No, it's not available on gist.github.com.<p>Sure, they may (or may not) port it over after some time, but I just don't get why they can't use the same code in the first place. Why is GitHub reinventing the wheel in for its very own sub-site?<p>(Other than that, this is absolutely great!)
Nice! I've been wanting this feature for a long time:<p><a href="http://philfreo.com/blog/the-problem-with-pull-requests-code-reviews/" rel="nofollow">http://philfreo.com/blog/the-problem-with-pull-requests-code...</a>
Since we're on the subject of Github's diff viewer, thought I'd throw in a plug for a chrome extension I wrote recently to improve it:<p><a href="https://chrome.google.com/webstore/detail/github-enhancement-suite/ahamcncifjblaomhphpfpopppadboiin" rel="nofollow">https://chrome.google.com/webstore/detail/github-enhancement...</a><p>Has side-by-side + word wrap + context (somewhat nicer looking than Github's, although could be biased)<p>Largely inspired by Phabricator's diff viewer.
Cool! Now the only thing missing is to integrate this[1] to natively support syntax highlight on diffs ;)<p>[1] <a href="https://github.com/danielribeiro/github-diff-highlight-extension" rel="nofollow">https://github.com/danielribeiro/github-diff-highlight-exten...</a>
I was just looking for this over the weekend, and found it this morning.<p>When I talk about code to collaborators in email or chat, I always try to link to the file on github rather than just giving a path. I much prefer to recieve feedback that way. Maybe i'm not good enough with my text editor, but it is just a lot faster to get to the file that way.
Been needing this badly for reviewing changes to legacy code which tends to require a much larger context for understanding of a diff.<p>Hope it shows up in GH enterprise soon!
phabricator has had this feature forever. Here's an example diff: <a href="https://secure.phabricator.com/rP1b026fa6291fd86ed6a2e598e6a013b5b6124532" rel="nofollow">https://secure.phabricator.com/rP1b026fa6291fd86ed6a2e598e6a...</a>
If any githubbers are reading, it would be great to be able to filter my organization's timeline events out of my personal timeline events. It's been bugging me for years. Thanks.