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
Unknown Object (File)
Thu, Apr 25, 1:13 AM
Unknown Object (File)
Tue, Apr 16, 2:47 PM
Unknown Object (File)
Tue, Apr 16, 2:47 PM
Unknown Object (File)
Fri, Apr 5, 1:04 PM
Unknown Object (File)
Mar 20 2024, 4:09 PM
Unknown Object (File)
Feb 20 2024, 6:08 PM
Unknown Object (File)
Feb 7 2024, 10:45 PM
Unknown Object (File)
Feb 3 2024, 8:02 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