Page MenuHomePhabricator

Consolidate outbound mail status in a new class
ClosedPublic

Authored by epriestley on Aug 13 2015, 6:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 20, 2:10 PM
Unknown Object (File)
Wed, Nov 20, 2:10 PM
Unknown Object (File)
Wed, Nov 20, 2:10 PM
Unknown Object (File)
Wed, Nov 20, 2:10 PM
Unknown Object (File)
Wed, Nov 20, 2:10 PM
Unknown Object (File)
Wed, Nov 20, 12:34 PM
Unknown Object (File)
Sat, Nov 16, 10:24 AM
Unknown Object (File)
Tue, Nov 12, 10:52 PM
Subscribers
None

Details

Summary

Ref T5791. This collects outbound mail status in one place and makes the list view a little spiffier.

Test Plan

Looked at list and detail views. Grepped for changed classes/constants.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Consolidate outbound mail status in a new class.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 13 2015, 6:48 PM
src/applications/metamta/constants/PhabricatorMailOutboundStatus.php
27

fa-warning?

28

fa-send?

29

fa-trash?

I played with this a little bit today, and it was awesome. Not sure the tabs are needed, do you think more this page's information will continue to grow?

There's at least a bit more stuff coming, yeah (at a minimum: exposing mail tags more clearly, adding Herald routing information).

I also imagine this may start looking and working a little more like your Gmail inbox eventually, where some users might read some of their mail directly from the web UI, so I'm trying to bend it in that direction a bit. I think the primary use case for this UI will be "understand what happened to mail", at least for a while, but I'd like to see if "read mail" can also eventually be a major use case. Stuff like "mail headers" is a little bit helpful in understanding what happened to mail, but totally useless when reading mail, which is why I moved it to a separate tab and tried to make the main view look about as much like an inbox message as it can.

epriestley edited edge metadata.
  • Use different icons for different statuses.
This revision was automatically updated to reflect the committed changes.