Page MenuHomePhorge

Curtain: add Hovercard support
Needs ReviewPublic

Authored by valerio.bozzolan on Mon, Aug 28, 13:19.
Tags
None
Referenced Files
F358314: D25424.diff
Tue, Sep 26, 06:55
Unknown Object (File)
Thu, Sep 21, 09:45
Unknown Object (File)
Sun, Sep 17, 17:55
Unknown Object (File)
Sat, Sep 16, 11:30
Unknown Object (File)
Tue, Sep 12, 05:51
Unknown Object (File)
Sun, Sep 10, 08:59
F342785: Curtain with Hovercard.png
Mon, Aug 28, 13:19

Details

Reviewers
None
Group Reviewers
O1: Blessed Committers
Maniphest Tasks
T15577: Show existing Hovercards from more places
Summary

Before this change, the right navigation panel (Curtain) had info without Hovercard support.

After this change, all fields, like Subscribers, Tags, Referenced Files etc. have Hovercards.

If you meet an UX problem with a specific field with Hovercard, you can disable that specific
Hovercard, calling PHUICurtainObjectRefView::setHovercarded(false).

This is an example, now with Hovercard support:

Curtain with Hovercard.png (830×1 px, 100 KB)

Closes T15577

Test Plan

No nuclear implosions when, on desktop and with a mouse:

Browse Maniphest Task and mouse hover Authored By, Assigned To, Subscribers, Referenced Files, ...

Browse Phriction document and mouse Tags, Referenced Files, Subscribers, ...

Browse single commit and mouse hover Referenced Files, Subscribers, ...

Browse Ponder question and mouse hover Tags, Referenced Files, Subscribers, ...

Browse Differential revision and mouse hover Tags, Referenced Files, Subscribers, ...

Diff Detail

Repository
rP Phorge
Branch
details-hovercarded
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 814
Build 814: arc lint + arc unit