Differential D21553

Restructure Hovercards to support more context information
ClosedPublic
Actions

Authored by epriestley on Feb 13 2021, 7:59 PM.

Tags

None

Referenced Files

	F13281042: D21553.diff
	Sun, Jun 2, 9:39 AM

	F13271413: D21553.diff
	Wed, May 29, 10:56 PM

	F13242581: D21553.diff
	Thu, May 23, 2:53 AM

	F13220903: D21553.diff
	Sun, May 19, 2:04 AM

	F13211594: D21553.id51310.diff
	Fri, May 17, 5:54 AM

	F13203437: D21553.diff
	Tue, May 14, 11:44 PM

	F13191309: D21553.diff
	Sat, May 11, 7:21 PM

	F13189019: D21553.diff
	Sat, May 11, 5:40 AM

View All 54 Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13602: Improve workflow when users who do not have permission to see a revision are added as reviewers
Commits: rP2aac3156f791: Restructure Hovercards to support more context information

Summary

Ref T13602. Currently, Hovercards are functions only of the object they represent (and the viewer, etc).

Recent changes to how users who can't see an object are rendered motivate making them a function of both the object they represent and the context in which they are being viewed. In particular, this enables a hovecard for a user to explain "This user can't see the thing you're lookign at right now.", so visual "exiled" markers can have a path forward toward discovery.

Test Plan

This change isn't expected to affect any behavior.
Viewed hovercards, moused over/out, resized windows, viewed standalone cards, viewed debug cards, saw no behavioral changes.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 13 2021, 7:59 PM

Harbormaster completed remote builds in B25113: Diff 51304.Feb 13 2021, 8:01 PM

epriestley requested review of this revision.Feb 13 2021, 8:01 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 13 2021, 9:37 PM

Closed by commit rP2aac3156f791: Restructure Hovercards to support more context information (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP2aac3156f791: Restructure Hovercards to support more context information.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

1 line

src/

aphront/

AphrontRequest.php

37 lines

applications/

search/

controller/

PhabricatorSearchHovercardController.php

40 lines

system/

events/

PhabricatorSystemDebugUIEventListener.php

2 lines

webroot/

rsrc/

js/

core/

169 lines

HovercardList.js

226 lines

behavior-hovercard.js

68 lines

Diff 51310

resources/celerity/map.php

Loading...

resources/celerity/packages.php

Loading...

src/aphront/AphrontRequest.php

Loading...

src/applications/search/controller/PhabricatorSearchHovercardController.php

Loading...

src/applications/system/events/PhabricatorSystemDebugUIEventListener.php

Loading...

webroot/rsrc/js/core/Hovercard.js

Loading...

webroot/rsrc/js/core/HovercardList.js

Loading...

webroot/rsrc/js/core/behavior-hovercard.js

Loading...