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

Only show text wrap and diff notes for text in merge requests. #7898

Merged
merged 1 commit into from Sep 29, 2014

Conversation

3 participants
@cirosantilli
Contributor

cirosantilli commented Sep 29, 2014

Before this PR those options also showed for binary files for which they don't apply:

screenshot from 2014-09-29 14 46 31 wrap text diff comments binary

Now they only show for text files.

@TeatroIO

This comment has been minimized.

TeatroIO commented Sep 29, 2014

I've prepared a stage. Click to open.

dzaporozhets added a commit that referenced this pull request Sep 29, 2014

Merge pull request #7898 from cirosantilli/hide-wrap-text-mr-binary
Only show text wrap and diff notes for text in merge requests.

@dzaporozhets dzaporozhets merged commit 3dc9a67 into gitlabhq:master Sep 29, 2014

1 check failed

default The build failed on Semaphore.
Details
@dzaporozhets

This comment has been minimized.

Member

dzaporozhets commented Sep 29, 2014

thanks

@cirosantilli cirosantilli deleted the cirosantilli:hide-wrap-text-mr-binary branch Sep 29, 2014

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