Page MenuHomePhabricator
Differential D9133

Use font icons on user approval queue
ClosedPublic
Actions

Authored by epriestley on May 15 2014, 1:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Feb 3, 2:17 AM
Unknown Object (File)
Sat, Feb 1, 11:49 AM
Unknown Object (File)
Tue, Jan 21, 9:00 AM
Unknown Object (File)
Mon, Jan 20, 10:48 AM
Unknown Object (File)
Sat, Jan 18, 6:15 AM
Unknown Object (File)
Sat, Jan 18, 6:04 AM
Unknown Object (File)
Thu, Jan 16, 1:26 AM
Unknown Object (File)
Jan 8 2025, 1:50 AM
View All Files
Subscribers
epriestley
shadowhand

Details

Reviewers
btrahan
shadowhand
chad
Maniphest Tasks
T5066: User queue approve/disapprove buttons are missing icons
Commits
Restricted Diffusion Commit
rP7f22958a8235: Use font icons on user approval queue
Summary

Fixes T5066.

Test Plan

Screen_Shot_2014-05-15_at_6.39.49_AM.png (1×1 px, 146 KB)

Diff Detail

Repository
rP Phabricator
Branch
approfont
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 482
Build 482: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 21694.May 15 2014, 1:40 PM
epriestley retitled this revision from to Use font icons on user approval queue.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
epriestley added a task: T5066: User queue approve/disapprove buttons are missing icons.
Herald added a subscriber: epriestley. · View Herald TranscriptMay 15 2014, 1:40 PM
Harbormaster completed remote builds in B482: Diff 21694.May 15 2014, 1:41 PM
shadowhand accepted this revision.May 15 2014, 1:48 PM
shadowhand added a reviewer: shadowhand.
shadowhand added a subscriber: shadowhand.

Looks right. Can't test locally because I don't have any users in queue ATM.

This revision is now accepted and ready to land.May 15 2014, 1:48 PM
epriestley closed this revision.May 15 2014, 1:51 PM
epriestley updated this revision to Diff 21695.

Closed by commit rP7f22958a8235 (authored by @epriestley).

chad edited edge metadata.May 15 2014, 2:01 PM

thx u :3

epriestley added a comment.May 15 2014, 2:01 PM

nps :3

Revision Contents
Changeset List

PathSize
src/
applications/
people/
controller/
4 lines

Diff 21694

View Options

src/applications/people/controller/PhabricatorPeopleListController.php

Loading...
Phabricator · Built by Phacility