Page MenuHomePhabricator
Differential D14879

Keep hovercards mostly on screen
ClosedPublic
Actions

Authored by epriestley on Dec 24 2015, 8:00 PM.
Tags
None
Referenced Files
F13239939: D14879.id.diff
Wed, May 22, 8:16 AM
F13216653: D14879.id35965.diff
Sat, May 18, 1:49 AM
F13211479: D14879.diff
Fri, May 17, 5:47 AM
F13209833: D14879.diff
Fri, May 17, 2:24 AM
F13196713: D14879.diff
Sun, May 12, 11:30 PM
F13175568: D14879.diff
Wed, May 8, 5:56 AM
Unknown Object (File)
Fri, May 3, 6:37 AM
Unknown Object (File)
Thu, May 2, 10:33 AM
View All Files
Subscribers
None
Tokens
"Like" token, awarded by avivey.

Details

Reviewers
chad
Maniphest Tasks
T8980: Make hovecards activate for standard handle lists
Commits
Restricted Diffusion Commit
rPe88dbbe1b1bc: Keep hovercards mostly on screen
Summary

Ref T8980.

Test Plan

Screen Shot 2015-12-24 at 11.59.28 AM.png (1×1 px, 211 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 35964.Dec 24 2015, 8:00 PM
epriestley retitled this revision from to Keep hovercards mostly on screen.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T8980: Make hovecards activate for standard handle lists.
chad accepted this revision.Dec 24 2015, 8:07 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 24 2015, 8:07 PM
epriestley updated this revision to Diff 35965.Dec 24 2015, 8:09 PM
epriestley edited edge metadata.
  • BONUS! Also keep it from going off the right side of the screen.
chad added a comment.Dec 24 2015, 8:09 PM

(y)

epriestley added a comment.Dec 24 2015, 8:10 PM

Yell if you figure out how to trick it into going offscreen. I think I got 'em all but maybe I missed something.

Closed by commit rPe88dbbe1b1bc: Keep hovercards mostly on screen (authored by epriestley, committed by epriestley). · Explain WhyDec 24 2015, 8:18 PM
This revision was automatically updated to reflect the committed changes.
avivey awarded a token.Dec 24 2015, 10:22 PM
avivey mentioned this in T10687: jx-tooltip does whatever it feels like.Mar 31 2016, 1:34 AM

Revision Contents
Changeset List

PathSize
resources/
celerity/
38 lines
webroot/
rsrc/
js/
core/
46 lines
28 lines

Diff 35968

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/js/core/Hovercard.js

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility