Page MenuHomePhabricator

Make the new "Unverified Email" behavior more clear to other users
ClosedPublic

Authored by epriestley on Feb 17 2017, 3:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 17, 10:03 PM
Unknown Object (File)
Sun, Apr 7, 10:13 PM
Unknown Object (File)
Fri, Mar 29, 3:22 PM
Unknown Object (File)
Thu, Mar 21, 8:37 AM
Unknown Object (File)
Mar 19 2024, 5:44 PM
Unknown Object (File)
Mar 19 2024, 5:44 PM
Unknown Object (File)
Mar 19 2024, 5:44 PM
Unknown Object (File)
Mar 19 2024, 5:44 PM
Subscribers
None

Details

Summary

Ref T12268. Ref T12157. When you mention or interact with a user who is unlikely to be able to respond (for example, because their account is disabled), we try to show a colored dot to provide a hint about this.

Recently, we no longer send any normal mail to unverified addresses. However, the rules for showing a dot haven't been updated yet, so they only care about this if auth.require-verification is set. This can be misleading, because if you say Hey @alice, what do you think about this? and she hasn't verified her email, you may not get a response.

Update the rule so users with unverified email addresses get a grey dot in all cases. The hint is basically "you shouldn't expect a response from this user".

Make the meaning of this hint more clear on the hovercard and profile.

Also:

  • Allow the non-ajax version of the hovercard page (which is basically only useful for testing hovercards) accept ?names=... so you can just plug usernames, hashtags, etc., in there.
  • Fix a bug where the user's join date was based on their profile creation date instead of account creation date on the hovercard. Users may not have a profile creation date (if they never changed any account details), and it may be different from their account creation date.
Test Plan

Screen Shot 2017-02-17 at 7.13.13 AM.png (1×1 px, 227 KB)

Diff Detail

Repository
rP Phabricator
Branch
email1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15675
Build 20702: Run Core Tests
Build 20701: arc lint + arc unit

Event Timeline

This changes the default style (when a user has no special attributes) a little bit by using setTag() on the header and causing it to render with a full tag/background/shade. I think it looks pretty good as-is and wanted to get the color there for "Disabled", but happy to tweak design stuff if you don't love it.

can we fix "Availability" too? Is became misaligned sometime with calendar changes.

I was planning on redesigning the user cards, so ship this if you want all these features and I'll clean it up next week.

This revision is now accepted and ready to land.Feb 17 2017, 5:21 PM

Yeah, I punted on "Availability" since I figure Hovercards in general are probably due for an update and I wasn't quite sure what to do about it.

This revision was automatically updated to reflect the committed changes.