Page MenuHomePhabricator

Move "hover over an inline to see the affected lines" code to the new class tree
ClosedPublic

Authored by epriestley on May 16 2017, 11:06 PM.
Tags
None
Referenced Files
F19887542: D17919.id.diff
Mon, Mar 23, 7:13 AM
F19826830: D17919.id.diff
Mon, Mar 9, 2:08 AM
F19826827: D17919.id.diff
Mon, Mar 9, 2:05 AM
F19826825: D17919.id.diff
Mon, Mar 9, 2:05 AM
F19825118: D17919.id43107.diff
Sun, Mar 8, 7:48 AM
F19750003: D17919.id.diff
Feb 15 2026, 9:05 PM
F19560155: D17919.id43107.diff
Jan 31 2026, 6:32 AM
F19560154: D17919.id43107.diff
Jan 31 2026, 6:31 AM
Subscribers
None

Details

Summary

Fixes T8047. Ref T12616. Fixes T9270. This moves the "hover" part of the hover/drag behavior to the new code, leaving the "drag" part for a followup change.

The new hover UI behaves properly with Quicksand (T8047) and the filetree (T9270).

Test Plan

Hovered over inlines, saw lines select properly.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Prior to this change, we also highlighted the <th /> ("Table Holumn") nodes with yellow. I think we don't really need to, although we could restore that once I finish up here. We could also make the overlay reticle bigger: I think its current size is to let you click the line numbers, but pointer-events: none does the same thing.

This revision is now accepted and ready to land.May 16 2017, 11:35 PM
This revision was automatically updated to reflect the committed changes.