Page MenuHomePhabricator

Give application actors in feed reasonable icons
ClosedPublic

Authored by epriestley on Apr 20 2016, 9:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 24, 5:26 AM
Unknown Object (File)
Sun, Mar 17, 12:20 PM
Unknown Object (File)
Sun, Mar 17, 12:20 PM
Unknown Object (File)
Sun, Mar 17, 12:12 PM
Unknown Object (File)
Sun, Mar 17, 11:28 AM
Unknown Object (File)
Thu, Mar 14, 4:05 PM
Unknown Object (File)
Fri, Mar 1, 12:58 AM
Unknown Object (File)
Feb 21 2024, 7:22 AM
Subscribers
None
Tokens
"Piece of Eight" token, awarded by cburroughs.

Details

Summary

Ref T8952. Currently, when an application (most commonly Herald, but sometimes Drydock, Diffusion, etc) publishes a feed story, we get an empty grey box for it in feed.

Instead, give the story a little application icon kind of "profile picture"-like thing.

Test Plan

Here's how it looks:

Screen Shot 2016-04-20 at 2.24.07 PM.png (153×362 px, 13 KB)

Feel free to tweak/counter-diff.

Diff Detail

Repository
rP Phabricator
Branch
feed1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11874
Build 14910: Run Core Tests
Build 14909: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Give application actors in feed reasonable icons.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
chad added inline comments.
webroot/rsrc/css/phui/phui-feed-story.css
19

can probably kill this now if everything has an icon/image

This revision is now accepted and ready to land.Apr 20 2016, 9:36 PM

I think (?) it looks better with it -- at least, the application icons do -- but feel free to shoot me an update if you want to nuke it.

This revision was automatically updated to reflect the committed changes.