Page MenuHomePhabricator

Use "fa-shopping-bag" instead of "fa-list-alt" for Owners package icon
ClosedPublic

Authored by epriestley on May 13 2016, 5:45 PM.

Details

Summary

Ref T10939. These appear in "Subscribers" tokenizers now and we got a maybe slightly better icon in the last FA update: instead of .

(I don't feel strongly about this, the old icon just doesn't seem very evocative.)

Test Plan

o.( O___O ).o

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley updated this revision to Diff 38317.May 13 2016, 5:45 PM
epriestley retitled this revision from to Use "fa-shopping-bag" instead of "fa-list-alt" for Owners package icon.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad accepted this revision.May 13 2016, 7:29 PM
chad edited edge metadata.

I thought it was , though maybe thats Owners in general.

This revision is now accepted and ready to land.May 13 2016, 7:29 PM

Ah, yeah, I think so. We should possibly make them consistent at some point -- we probably need something similar for T5055 eventually too, although maybe we'll get another round of FA updates by then.

This revision was automatically updated to reflect the committed changes.