Refined-github: Add opt-in setting to remove diff signs

[jerone]

There is (or was) a feature called remove-diff-signs.js, but it's intention is to make the diff signs unselectable. Personally I think those signs are redundant in combination with the background colors in the diffs and can be removed.

If this feature request is approved, then the following code is probably all you need to include:

.blob-code-marker-addition:before,
.blob-code-marker-context:before,
.blob-code-marker-deletion:before {
    display: none !important;
}

[fregante]

those signs are redundant in combination with the background colors

Fun fact: the feature was actually called "remove-diff-signs" because that's exactly what its first version did. (https://github.com/sindresorhus/refined-github/pull/254) However the original PR was later changed because relying on colors made it less accessible. https://github.com/sindresorhus/refined-github/pull/254#issuecomment-228229599

I'd be fine with that but others might not. Either way since it's just simple CSS you can just place that in the options rather than add it as a default.

[sindresorhus]

I think this would be nice, but for accessibility reasons, it should be opt-in.

[sindresorhus]

Reopening as I think we should make this an opt-in setting.

[sindresorhus]

@bfred-it Are you still against this even if it's opt-in? I agree it's not a good default for accessibility reasons, but I personally find the plus/minus moot as the colors are enough for me, so I'd like to opt into hiding those, as they just create visual noise.

[fregante]

Opt-out can be done. Opt-in not so much :)