Improve commit audit status icons
ClosedPublic
Actions

Authored by epriestley on Jan 13 2017, 12:33 AM.

Tags

None

Referenced Files

	F22750864: D17203.id.diff
	Sun, Oct 4, 10:10 PM

	F22684844: D17203.id41364.diff
	Fri, Oct 2, 4:39 AM

	F22684840: D17203.id41363.diff
	Fri, Oct 2, 4:39 AM

	F22684839: D17203.id.diff
	Fri, Oct 2, 4:39 AM

	F22678877: D17203.diff
	Fri, Oct 2, 12:27 AM

	F22565446: D17203.id41363.diff
	Fri, Sep 25, 12:02 PM

	Unknown Object (File)
	Tue, Sep 15, 4:19 PM

	Unknown Object (File)
	Sep 11 2026, 10:20 AM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T9482: Problems in the "Commits in this Package that Need Attention" view

Commits

rP4d0a03e3d0b2: Improve commit audit status icons

Summary

Ref T9482. These may need a little more work (feel free to shoot me a counter-diff) but try to:

Never use only color to distinguish between states (for colorblind, etc users).
Give the "nothing needs to be done" state a more obvious "okay" icon (instead of a question mark).

Test Plan

Looked at some linked commits in Maniphest, the icons made a bit more sense?

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 13 2017, 12:33 AM

chad accepted this revision.Jan 13 2017, 12:34 AM

This revision is now accepted and ready to land.Jan 13 2017, 12:34 AM

Harbormaster completed remote builds in B15251: Diff 41363.Jan 13 2017, 12:34 AM

Closed by commit rP4d0a03e3d0b2: Improve commit audit status icons (authored by epriestley, committed by epriestley). · Explain WhyJan 13 2017, 12:35 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

audit/

constants/

PhabricatorAuditCommitStatusConstants.php

14 lines

Diff 41364

View Options

src/applications/audit/constants/PhabricatorAuditCommitStatusConstants.php

Improve commit audit status iconsClosedPublicActions