Page MenuHomePhabricator

Update diff highlight colors for better color blindess distinction
ClosedPublic

Authored by chad on Jan 4 2017, 7:22 PM.
Tags
None
Referenced Files
F18044734: D17138.id41215.diff
Sun, Aug 3, 7:42 AM
F17922181: D17138.diff
Wed, Jul 30, 6:25 AM
F17884732: D17138.id.diff
Tue, Jul 29, 12:06 AM
F17860942: D17138.diff
Sun, Jul 27, 10:22 PM
Unknown Object (File)
Jun 18 2025, 6:18 PM
Unknown Object (File)
Jun 16 2025, 11:01 AM
Unknown Object (File)
May 29 2025, 8:55 AM
Unknown Object (File)
May 9 2025, 12:35 AM
Subscribers

Details

Summary

Tweaks the diff colors here a bit, as well as making full diffs slightly easier to read in full. Ref T12060

Test Plan

Tested prose diffs, email prose diffs, and a regular Differential revision.

pasted_file (1×2 px, 371 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jan 4 2017, 7:23 PM
This revision was automatically updated to reflect the committed changes.

I can barely see the light green shade now, but maybe just a monitor issue unique to me. Light red is OK.

Well, maybe "barely see" is a little dramatic. But it's light enough that scanning the right-hand-side of two-up diffs for new blocks of code feels harder than it used to be.

I changed new-full to be light instead of bright, maybe that's it?

(it felt too overpowering with a richer green)

I can bring light green up a hair