Page MenuHomePhabricator
Differential D13059

[Redesign] Remove remaining barColor callsites
ClosedPublic
Actions

Authored by chad on May 28 2015, 10:15 PM.
Tags
None
Referenced Files
F19825578: D13059.id.diff
Sun, Mar 8, 12:54 PM
F19825577: D13059.id.diff
Sun, Mar 8, 12:54 PM
F19785991: D13059.id.diff
Wed, Feb 25, 4:35 AM
F19503509: D13059.id31507.diff
Jan 9 2026, 12:16 AM
F19092596: D13059.id.diff
Dec 3 2025, 5:54 PM
F19085512: D13059.diff
Dec 2 2025, 6:37 PM
F19061840: D13059.diff
Nov 29 2025, 1:46 PM
F19024138: D13059.diff
Nov 24 2025, 2:28 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
T8341: Identify anywhere we didn't use icons where barColor was used
T8099: Mid-2015 Phabricator Redesign
Commits
Restricted Diffusion Commit
rP13f9f209df06: [Redesign] Remove remaining barColor callsites
Summary

Ref T8099, Fixes T8341. Switches all remaining callsites to setBarColor to use setStatusIcon (sans workboards).

Test Plan

Test each of the applications I changed as I could (not Releeph).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 31507.May 28 2015, 10:15 PM
chad retitled this revision from to [Redesign] Remove remaining barColor callsites.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad added tasks: T8099: Mid-2015 Phabricator Redesign, T8341: Identify anywhere we didn't use icons where barColor was used.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMay 28 2015, 10:15 PM
epriestley accepted this revision.May 28 2015, 10:16 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 28 2015, 10:16 PM
chad added a comment.May 28 2015, 10:17 PM

oh dear, ponder is looking better.

Closed by commit rP13f9f209df06: [Redesign] Remove remaining barColor callsites (authored by chad, committed by chad). · Explain WhyMay 28 2015, 10:17 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
audit/
view/
8 lines
harbormaster/
query/
3 lines
phortune/
controller/
3 lines
4 lines
phrequent/
query/
2 lines
ponder/
constants/
10 lines
query/
10 lines
releeph/
controller/
product/
2 lines
infrastructure/
customfield/
config/
2 lines

Diff 31508

View Options

src/applications/audit/view/PhabricatorAuditListView.php

Loading...
View Options

src/applications/harbormaster/query/HarbormasterBuildableSearchEngine.php

Loading...
View Options

src/applications/phortune/controller/PhortuneAccountViewController.php

Loading...
View Options

src/applications/phortune/controller/PhortuneMerchantViewController.php

Loading...
View Options

src/applications/phrequent/query/PhrequentSearchEngine.php

Loading...
View Options

src/applications/ponder/constants/PonderQuestionStatus.php

Loading...
View Options

src/applications/ponder/query/PonderQuestionSearchEngine.php

Loading...
View Options

src/applications/releeph/controller/product/ReleephProductViewController.php

Loading...
View Options

src/infrastructure/customfield/config/PhabricatorCustomFieldConfigOptionType.php

Loading...
Phabricator · Built by Phacility