Page MenuHomePhabricator

Make the "Requested Changes to Prior Diff" reviewer icon red, not bluegrey
ClosedPublic

Authored by epriestley on Aug 10 2017, 3:21 PM.
Tags
None
Referenced Files
F15527814: D18385.id44186.diff
Tue, Apr 22, 6:43 AM
F15522242: D18385.id44192.diff
Sun, Apr 20, 6:58 PM
F15514777: D18385.diff
Fri, Apr 18, 5:07 AM
F15472303: D18385.id44192.diff
Sat, Apr 5, 12:35 PM
F15462387: D18385.id44186.diff
Tue, Apr 1, 2:47 PM
F15424070: D18385.diff
Mar 22 2025, 8:02 PM
F15419558: D18385.id44186.diff
Mar 21 2025, 7:02 AM
F15418167: D18385.id44192.diff
Mar 20 2025, 9:17 PM
Subscribers

Details

Summary

See PHI31. The "Accepted Older Revision" icon is (more reasonably) bluegrey, but that rule spilled over here where it doesn't make much sense. "Requested Changes to Prior Diff" remains in effect across updates, but the coloration implies otherwise.

Test Plan

"Requested Changes to This Diff" (unchanged):

Screen Shot 2017-08-10 at 8.18.21 AM.png (198×422 px, 20 KB)

"Requested Changes to Prior Diff" (now red, previously bluegrey):

Screen Shot 2017-08-10 at 8.18.47 AM.png (242×457 px, 27 KB)

Note that the icons are different so this is technically colorblind-safe, and it's normally not important to distinguish between these two reds anyway.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable