Differential D17171

Re-darken full width diff colors
ClosedPublic
Actions

Authored by chad on Jan 10 2017, 9:14 PM.

Tags

None

Referenced Files

	F18834335: D17171.diff
	Sun, Oct 26, 8:02 AM

	F18833680: D17171.diff
	Sun, Oct 26, 5:06 AM

	F18825333: D17171.diff
	Thu, Oct 23, 9:06 PM

	F18735015: D17171.id41297.diff
	Tue, Sep 30, 11:09 PM

	F18651881: D17171.id41298.diff
	Sep 21 2025, 5:52 AM

	F18642843: D17171.id41298.diff
	Sep 19 2025, 12:55 AM

	F18642842: D17171.id.diff
	Sep 19 2025, 12:55 AM

	F18627773: D17171.id41298.diff
	Sep 16 2025, 4:44 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T12089: New red/green highlight colors are incorrect for added or removed new lines

Commits

rP63af2275ca05: Re-darken full width diff colors

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Jan 10 2017, 9:14 PM

Herald added a subscriber: Korvin. · View Herald TranscriptJan 10 2017, 9:14 PM

chad edited the test plan for this revision. (Show Details)Jan 10 2017, 9:15 PM

Harbormaster completed remote builds in B15184: Diff 41297.Jan 10 2017, 9:15 PM

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.

Closed by commit rP63af2275ca05: Re-darken full width diff colors (authored by chad, committed by chad). · Explain WhyJan 10 2017, 9:25 PM

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).

epriestley mentioned this in T12089: New red/green highlight colors are incorrect for added or removed new lines.Jan 10 2017, 9:48 PM

Revision Contents
Changeset List

Path

Size

resources/

celerity/

12 lines

webroot/

rsrc/

css/

application/

differential/

changeset-view.css

10 lines

Diff 41298

resources/celerity/map.php

Loading...

webroot/rsrc/css/application/differential/changeset-view.css

Loading...