Page MenuHomePhorge

Curtain: add Hovercard support
ClosedPublic

Authored by valerio.bozzolan on Aug 28 2023, 13:19.
Tags
None
Referenced Files
F3748606: D25424.1745778608.diff
Sat, Apr 26, 18:30
F3731136: D25424.1745736215.diff
Sat, Apr 26, 06:43
F3717094: D25424.1745694218.diff
Fri, Apr 25, 19:03
F3592637: D25424.1745145932.diff
Sat, Apr 19, 10:45
F3577616: D25424.1745087481.diff
Fri, Apr 18, 18:31
F3554373: D25424.1744987775.diff
Thu, Apr 17, 14:49
F3485539: D25424.1744776050.diff
Tue, Apr 15, 04:00
F3449670: D25424.1744711569.diff
Mon, Apr 14, 10:06

Details

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
Lint
Lint Not Applicable
Unit
Tests Not Applicable