Page MenuHomePhabricator

Render user hovercards with context information about their ability to see the context object
ClosedPublic

Authored by epriestley on Feb 13 2021, 8:56 PM.
Tags
None
Referenced Files
F13278416: D21554.id.diff
Fri, May 31, 9:02 PM
F13260692: D21554.diff
Mon, May 27, 12:15 AM
F13244831: D21554.diff
Thu, May 23, 5:21 AM
F13198689: D21554.id.diff
Mon, May 13, 8:53 AM
F13198530: D21554.id51306.diff
Mon, May 13, 7:31 AM
F13196431: D21554.diff
Sun, May 12, 11:15 PM
F13184470: D21554.id51305.diff
Fri, May 10, 7:07 PM
Unknown Object (File)
Mon, May 6, 3:45 PM
Subscribers
None

Details

Summary

Ref T13602. When rendering a user hovercard, pass the object on which the reference appears. If the user can't see the object, make it clear on the hovecard.

Restyle the "nopermission" markup in mentions to make it more obvious what the style means: instead of grey text, use red with an explicit icon.

Test Plan

Screen Shot 2021-02-13 at 12.50.53 PM.png (467×564 px, 35 KB)

Diff Detail

Repository
rP Phabricator
Branch
policy5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25115
Build 34658: Run Core Tests
Build 34657: arc lint + arc unit

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 13 2021, 8:57 PM
Harbormaster failed remote builds in B25114: Diff 51305!
This revision was not accepted when it landed; it landed in state Needs Review.Feb 13 2021, 9:38 PM
This revision was automatically updated to reflect the committed changes.