Page MenuHomePhabricator
Differential D17141

Bring up contrast on light green / red diffs
ClosedPublic
Actions

Authored by chad on Jan 4 2017, 10:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Feb 26, 3:30 AM
Unknown Object (File)
Fri, Feb 21, 3:01 PM
Unknown Object (File)
Sat, Feb 15, 10:14 PM
Unknown Object (File)
Feb 3 2025, 8:37 PM
Unknown Object (File)
Feb 3 2025, 8:00 PM
Unknown Object (File)
Feb 1 2025, 10:18 PM
Unknown Object (File)
Jan 31 2025, 3:04 PM
Unknown Object (File)
Jan 19 2025, 3:12 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP96fbf37dcc11: Bring up contrast on light green / red diffs
Summary

Minor color saturation here, ideal for low quality monitors.

Test Plan

Review new colors in various scenarios.

pasted_file (880×2 px, 163 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.Jan 4 2017, 10:37 PM
Herald added a subscriber: Korvin. · View Herald TranscriptJan 4 2017, 10:37 PM
epriestley accepted this revision.Jan 4 2017, 10:38 PM
epriestley edited edge metadata.

thx u

Harbormaster completed remote builds in B15106: Diff 41221.Jan 4 2017, 10:38 PM
chad planned changes to this revision.Jan 4 2017, 10:39 PM

actually going to spend some more time on this.

chad updated this revision to Diff 41223.Jan 4 2017, 11:06 PM
  • Stronger color hinting, but only on side of diffs
Harbormaster completed remote builds in B15108: Diff 41223.Jan 4 2017, 11:06 PM
chad edited the test plan for this revision. (Show Details)Jan 4 2017, 11:07 PM
Harbormaster completed remote builds in B15108: Diff 41223.Jan 4 2017, 11:07 PM
epriestley accepted this revision.Jan 4 2017, 11:07 PM
This revision is now accepted and ready to land.Jan 4 2017, 11:07 PM
Closed by commit rP96fbf37dcc11: Bring up contrast on light green / red diffs (authored by chad, committed by chad). · Explain WhyJan 4 2017, 11:18 PM
This revision was automatically updated to reflect the committed changes.
chad added a comment.Jan 4 2017, 11:24 PM

let me know

Revision Contents
Changeset List

PathSize
resources/
celerity/
12 lines
src/
applications/
differential/
render/
20 lines
4 lines
webroot/
rsrc/
css/
application/
differential/
10 lines

Diff 41226

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/differential/render/DifferentialChangesetOneUpRenderer.php

Loading...
View Options

src/applications/differential/render/DifferentialChangesetTwoUpRenderer.php

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility