Page MenuHomePhabricator
Differential D15154

Always give users "fa-user" icons in tokenizers
ClosedPublic
Actions

Authored by epriestley on Feb 1 2016, 5:46 PM.
Tags
None
Referenced Files
F18616196: D15154.id36586.diff
Sun, Sep 14, 4:33 PM
F18599923: D15154.id.diff
Sat, Sep 13, 8:35 AM
F18590528: D15154.diff
Fri, Sep 12, 6:19 AM
Unknown Object (File)
Jun 20 2025, 6:03 PM
Unknown Object (File)
May 25 2025, 6:57 AM
Unknown Object (File)
May 6 2025, 3:28 AM
Unknown Object (File)
May 6 2025, 12:44 AM
Unknown Object (File)
Apr 27 2025, 3:58 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10247: Don't show custom user icons in typeaheads
Commits
Restricted Diffusion Commit
rP18f34fab7397: Always give users "fa-user" icons in tokenizers
Summary

Fixes T10247. The flavor icons are unhelpful/confusing in these contexts; show a boringer icon instead.

Test Plan

Used tokenizer to select user with custom profile icon. Reloaded page. Saw boringer icon in both cases.

Diff Detail

Repository
rP Phabricator
Branch
uicon
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10428
Build 12742: Run Core Tests
Build 12741: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 36586.Feb 1 2016, 5:46 PM
epriestley retitled this revision from to Always give users "fa-user" icons in tokenizers.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10247: Don't show custom user icons in typeaheads.
chad accepted this revision.Feb 1 2016, 5:48 PM
chad edited edge metadata.

Mmmm Boringer Flavor.

Kent All-750x750.jpg (750×750 px, 669 KB)

This revision is now accepted and ready to land.Feb 1 2016, 5:48 PM
Closed by commit rP18f34fab7397: Always give users "fa-user" icons in tokenizers (authored by epriestley, committed by epriestley). · Explain WhyFeb 1 2016, 5:49 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
people/
phid/
6 lines
phid/
14 lines
typeahead/
view/
2 lines

Diff 36586

View Options

src/applications/people/phid/PhabricatorPeopleUserPHIDType.php

Loading...
View Options

src/applications/phid/PhabricatorObjectHandle.php

Loading...
View Options

src/applications/typeahead/view/PhabricatorTypeaheadTokenView.php

Loading...
Phabricator · Built by Phacility