Page MenuHomePhabricator
Differential D17145

Make JX.Tooltip more conservative about positioning
ClosedPublic
Actions

Authored by epriestley on Jan 5 2017, 4:52 PM.
Tags
None
Referenced Files
F19259250: D17145.diff
Mon, Dec 22, 3:20 PM
F19072964: D17145.id.diff
Dec 1 2025, 12:21 AM
F19058015: D17145.diff
Nov 29 2025, 12:34 AM
F18974065: D17145.diff
Nov 15 2025, 6:23 PM
F18769825: D17145.id41240.diff
Oct 8 2025, 9:25 AM
F18769613: D17145.id41232.diff
Oct 8 2025, 8:22 AM
F18627053: D17145.diff
Sep 16 2025, 3:04 AM
F18536430: D17145.diff
Sep 7 2025, 2:16 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10687: jx-tooltip does whatever it feels like
T12064: Tooltip for "Preview" in comment box overflows horizontally
Commits
rP5ff02058a4b0: Make JX.Tooltip more conservative about positioning
Summary

Fixes T10687. Fixes T12064.

  • Primarily, adds a margin around the edge of the screen for the purposes of aligning the tooltip.
  • Also, tries to flip the tooltip if it can (e.g., if the tooltip normally goes east, try west first), then tries other positions exhastively.
Test Plan

Screen Shot 2017-01-05 at 8.48.28 AM.png (207×226 px, 8 KB)

Screen Shot 2017-01-05 at 8.49.17 AM.png (266×318 px, 15 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 5 2017, 4:52 PM
Harbormaster completed remote builds in B15116: Diff 41232.Jan 5 2017, 4:54 PM
chad accepted this revision.Jan 5 2017, 5:01 PM
chad edited edge metadata.

8071526975_7bc933b835.jpg (375×500 px, 69 KB)

epriestley added a comment.Jan 5 2017, 5:03 PM

Curiously, this did not transition to "Accepted".

chad accepted this revision.Jan 5 2017, 5:05 PM
This revision is now accepted and ready to land.Jan 5 2017, 5:05 PM
chad added a comment.Jan 5 2017, 5:05 PM

frowncat

epriestley added a comment.Jan 5 2017, 5:07 PM

I think "Accept" + "Comment" may be interacting, probably not too tricky to fix.

chad added a comment.Jan 5 2017, 5:17 PM

11-490.png (480×640 px, 26 KB)

Closed by commit rP5ff02058a4b0: Make JX.Tooltip more conservative about positioning (authored by epriestley, committed by epriestley). · Explain WhyJan 6 2017, 12:03 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
18 lines
webroot/
rsrc/
js/
core/
109 lines

Diff 41240

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/js/core/ToolTip.js

Loading...
Phabricator · Built by Phacility