Page MenuHomePhabricator

Add a "View Revision" button to HTML email
ClosedPublic

Authored by chad on May 11 2016, 1:12 AM.

Details

Summary

Ref T10694. If this feels good, I'd plan to eventually add something similar to other applications ("View Task", etc).

Not sure if we should keep the object link later in the mail body or not. I left it for now.

Test Plan

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 retitled this revision from to Add a "View Revision" button to HTML email.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

I don't particularly love my design here, not sure if you have any ideas. Some things I might try:

  • Get rid of the background and bottom border?
  • Make the button a little "lighter weight" visually so it's more like a link than a button? I do want it to stand out a bit but using the "grey" style or something similar with a touch of color might feel better.

If we want to make html the default, I'll spend some time on it Monday, unless you want to ship something sooner.

Sounds good, definitely no rush. Might be a good time to play around with simplifying the footer a bit too (e.g., just a "Preferences" link).

chad edited reviewers, added: epriestley; removed: chad.
  • remove all personality

This is a bit bare bones, but feels decent on my desktop and phone.

epriestley edited edge metadata.

Lifeless!

This revision is now accepted and ready to land.May 18 2016, 9:23 PM

(I think simple is probably pretty good in email.)

I think i might use it actually, at least on mobile.

This revision was automatically updated to reflect the committed changes.

Of course doesn't render in my daily use mobile client. But Apple is fine.

I get this as a body summary too now ("View Revision" at the beginning), although there's probably nothing we can really do about that:

There's maybe more whitespace than we need here, too, between the button and the comment:

should be easy to fix those both.