Page MenuHomePhabricator

Re-darken full width diff colors
ClosedPublic

Authored by chad on Jan 10 2017, 9:14 PM.
Tags
None
Referenced Files
F12853889: D17171.id41297.diff
Fri, Mar 29, 8:28 AM
Unknown Object (File)
Thu, Mar 21, 4:22 AM
Unknown Object (File)
Wed, Mar 20, 6:09 PM
Unknown Object (File)
Sun, Mar 3, 7:32 AM
Unknown Object (File)
Feb 15 2024, 3:54 PM
Unknown Object (File)
Feb 13 2024, 9:17 PM
Unknown Object (File)
Feb 13 2024, 9:17 PM
Unknown Object (File)
Feb 13 2024, 9:17 PM
Subscribers

Details

Summary

Ref T12089. This reverts back to using "bright" colors for full width changes in green and red.

Test Plan

Review diffs, see full width colors.

pasted_file (1×3 px, 995 KB)

Diff Detail

Repository
rP Phabricator
Branch
diff-colors (branched from master)
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 15184
Build 19950: Run Core Tests
Build 19949: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jan 10 2017, 9:20 PM

this green has more contrast than the previous "bright" green, just FYI, which is why I made this change.

This revision was automatically updated to reflect the committed changes.

If this color scheme isn't good for red-green colorblind users we can introduce a setting, but this was definitely noticeably worse for me too (I just couldn't put my finger on why).